refactor: fetch batch messages on batch subscribe

WalletConnect · Nov 29, 2024 · a674347 · a674347
1 parent b3ca5f4
commit a674347
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/core/src/controllers/relayer.ts b/packages/core/src/controllers/relayer.ts
@@ -207,7 +207,7 @@ export class Relayer extends IRelayer {
         },
         "relayer.request - publishing...",
       );
-      const tag = `${id}:${(request.params as any)?.tag}`;
+      const tag = `${id}:${(request.params as any)?.tag || ""}`;
       this.requestsInFlight.push(tag);
       const result = await this.provider.request(request);
       this.requestsInFlight = this.requestsInFlight.filter((i) => i !== tag);

diff --git a/packages/core/src/controllers/subscriber.ts b/packages/core/src/controllers/subscriber.ts
@@ -535,7 +535,7 @@ export class Subscriber extends ISubscriber {
     this.pending.forEach((params) => {
       pendingSubscriptions.push(params);
     });
-
+    await this.batchFetchMessages(pendingSubscriptions);
     await this.batchSubscribe(pendingSubscriptions);
   };