fix(notifications): make sure appeal possible notifications unique

kleros · Mar 5, 2018 · 30ea202 · 30ea202
1 parent 957bf06
commit 30ea202
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/src/abstractWrappers/Disputes.js b/src/abstractWrappers/Disputes.js
@@ -121,7 +121,6 @@ class Disputes extends AbstractWrapper {
       notificationCallback = callback
     ) => {
       const newPeriod = event.args._period.toNumber()
-      const txHash = event.transactionHash
       // send appeal possible notifications
       if (newPeriod === arbitratorConstants.PERIOD.APPEAL) {
         this._checkArbitratorWrappersSet()
@@ -152,7 +151,7 @@ class Disputes extends AbstractWrapper {
             ) {
               const notification = await this._StoreProvider.newNotification(
                 address,
-                txHash,
+                arbitratorAddress, // use arbitratorAddress so that we know it is unique. not event based
                 disputeId, // use disputeId instead of logIndex since it doens't have its own event
                 notificationConstants.TYPE.APPEAL_POSSIBLE,
                 'A ruling has been made. Appeal is possible',