Merge pull request #307 from hayes-mysten/main

feat(kit): Only report effects if the wallet has implemented the reportTransactionEffects feature
suiet · Oct 16, 2024 · 2728867 · 2728867
2 parents 525c2ec + 3918fbc
commit 2728867
Showing 1 changed file with 10 additions and 8 deletions.
diff --git a/packages/kit/src/components/WalletProvider.tsx b/packages/kit/src/components/WalletProvider.tsx
@@ -347,14 +347,16 @@ export const WalletProvider = (props: WalletProviderProps) => {
         );
       }
 
-      try {
-        await _wallet.reportTransactionEffects({
-          effects: effectsB64,
-          account,
-          chain: chain.id as IdentifierString,
-        });
-      } catch (error) {
-        console.warn("Failed to report transaction effects:", error);
+      if (_wallet.hasFeature(FeatureName.SUI__REPORT_TRANSACTION_EFFECTS)) {
+        try {
+          await _wallet.reportTransactionEffects({
+            effects: effectsB64,
+            account,
+            chain: chain.id as IdentifierString,
+          });
+        } catch (error) {
+          console.warn("Failed to report transaction effects:", error);
+        }
       }
 
       return {