stop indexing send events

Author: kasparkallas

packages/subgraph/schema.graphql

@@ -1385,6 +1385,9 @@ type MintedEvent implements Event @entity(immutable: true) {
     operatorData: Bytes!
 }
 
+"""
+WARNING: This event has been deprecated. It is not indexed anymore. It's kept in schema to avoid breakage in the schema.
+"""
 type SentEvent implements Event @entity(immutable: true) {
     id: ID!
     transactionHash: Bytes!

packages/subgraph/src/mappingHelpers.ts

@@ -1082,9 +1082,6 @@ export function updateATSStreamedAndBalanceUntilUpdatedAt(
 ): void {
     const block = event.block;
 
-    // update the updatedAt property of the account that just made an update
-    updateAccountUpdatedAt(accountAddress, block);
-
     let accountTokenSnapshot = getOrInitAccountTokenSnapshot(
         accountAddress,
         tokenAddress,
@@ -1104,7 +1101,7 @@ export function updateATSStreamedAndBalanceUntilUpdatedAt(
 
     const balanceUntilUpdatedAtAfterUpdate = accountTokenSnapshot.balanceUntilUpdatedAt;
 
-    if (accountTokenSnapshot.updatedAtBlockNumber === block.number && balanceUntilUpdatedAtAfterUpdate.equals(balanceUntilUpdatedAtBeforeUpdate)) {
+    if (accountTokenSnapshot.updatedAtBlockNumber === block.number && balanceUntilUpdatedAtBeforeUpdate.equals(balanceUntilUpdatedAtAfterUpdate)) {
         // ATS has already been updated in the block and no new balance change discovered. It's safe to return early.
         return;
     }
@@ -1197,6 +1194,9 @@ export function updateATSStreamedAndBalanceUntilUpdatedAt(
         tokenStatistic.save();
     }
 
+    // update the updatedAt property of the account that just made an update
+    updateAccountUpdatedAt(accountAddress, block);
+
     _createAccountTokenSnapshotLogEntity(
         event,
         accountAddress,

packages/subgraph/subgraph.template.yaml

@@ -380,9 +380,10 @@ templates:
         - event: Minted(indexed address,indexed address,uint256,bytes,bytes)
           handler: handleMinted
           receipt: true
-        - event: Sent(indexed address,indexed address,indexed address,uint256,bytes,bytes)
-          handler: handleSent
-          receipt: true
+        # Not indexing Sent event anymore. Use Transfer instead.
+        # - event: Sent(indexed address,indexed address,indexed address,uint256,bytes,bytes)
+        #   handler: handleSent
+        #   receipt: true
         - event: TokenUpgraded(indexed address,uint256)
           handler: handleTokenUpgraded
           receipt: true

packages/subgraph/tests/superToken/event/superToken.event.test.ts

@@ -516,6 +516,26 @@ describe("SuperToken Mapper Unit Tests", () => {
                 to,
                 value
             );
+            transferEvent.block.timestamp = BigInt.fromI32(1);
+            transferEvent.block.number = BigInt.fromI32(1);
+
+            mockedRealtimeBalanceOf(
+                transferEvent.address.toHex(),
+                from,
+                transferEvent.block.timestamp,
+                value,
+                BIG_INT_ZERO,
+                BIG_INT_ZERO
+            );
+
+            mockedRealtimeBalanceOf(
+                transferEvent.address.toHex(),
+                to,
+                transferEvent.block.timestamp,
+                value,
+                BIG_INT_ZERO,
+                BIG_INT_ZERO
+            );
 
             handleTransfer(transferEvent);
 
@@ -564,6 +584,8 @@ describe("SuperToken Mapper Unit Tests", () => {
                 to,
                 value
             );
+            secondTransferEvent.block.timestamp = BigInt.fromI32(2);
+            secondTransferEvent.block.number = BigInt.fromI32(2);
 
             mockedRealtimeBalanceOf(
                 secondTransferEvent.address.toHex(),
@@ -574,6 +596,15 @@ describe("SuperToken Mapper Unit Tests", () => {
                 BIG_INT_ZERO
             );
 
+            mockedRealtimeBalanceOf(
+                secondTransferEvent.address.toHex(),
+                to,
+                secondTransferEvent.block.timestamp,
+                value.times(BigInt.fromI32(2)),
+                BIG_INT_ZERO,
+                BIG_INT_ZERO
+            );
+
             handleTransfer(secondTransferEvent);
 
             assertTokenStatisticProperties(