fix: use paging to fail all pending user transactions

[chmanie]

Description

There was a problem with pending transactions not being put into a failed state properly. This usually happens when a proper user wallet is initialized.

This PR uses Apollo paging to put all pending transactions into a failed state.

Testing

1. Stop your dev environment
2. Apply this patch (it will create 120 pending transactions during the create-data script.

diff --git a/scripts/create-data.js b/scripts/create-data.js
index f0ed7968e..f0a528dd9 100644
--- a/scripts/create-data.js
+++ b/scripts/create-data.js
@@ -300,6 +300,27 @@ const addTxToDb = async ({
   return graphqlRequest(createTransaction, { input }, GRAPHQL_URI, API_KEY);
 };
 
+const create120Transactions = async (userAddress, colonyAddress) => {
+  const domainBatchKey = 'createDomain';
+  for(let i=0; i<120; i++) {
+    const domainGroupId = nanoid();
+    console.log(`Adding pending transaction number ${i} with id ${domainGroupId}`);
+    await addTxToDb({
+      colonyAddress,
+      context: ClientType.ColonyClient,
+      groupId: domainGroupId,
+      groupIndex: 0,
+      groupKey: domainBatchKey,
+      hash: '0xdeadcafe',
+      methodName: 'addDomain(uint256,uint256,uint256)',
+      params: [1],
+      status: 'PENDING',
+      userAddress,
+    });
+    await delay(5);
+  }
+}
+
 /*
  * User
  */
@@ -1715,6 +1736,8 @@ const createUserAndColonyData = async () => {
     validatedTokensOwner
   );
 
+  await create120Transactions(leelaWallet.address, Object.keys(availableColonies)[0]);
+
   if (reputationMining) {
     console.log(
       "Reputation mining should now be disabled. Make sure you check this manually, otherwise you'll get chain time skips",

3. Restart your dev environment
4. Run the create-data script
5. Open the CDapp and see 120 failed transactions

Diffs

Changes 🏗

• Use paging to fail pending transactions on refresh

[jakubcolony]

Works great! Thanks for such a quick turnaround 👍

Screen.Recording.2024-08-01.at.23.58.17.mov

[jakubcolony]

You could also pass an increased limit here to override the default 100 items

[bassgeta]

Code changes look sane and the infinite scroll works!


Let's go 😎

[rdig]

Tested this and can confirm it's working as expected.

The patch to the create data script added the pending tx's correctly:

I also disabled the failPendingTransactions method, so that I can view the pending tx's in the UI:

And once I re-enabled it, the all changed their status to Failed, as expected:

Nicely done Chris!