fix(vue): optional chain ID when getting accounts (#277)

tien · Oct 20, 2024 · 127620f · 127620f
1 parent 5e63f82
commit 127620f
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/.changeset/olive-rats-join.md b/.changeset/olive-rats-join.md
@@ -0,0 +1,5 @@
+---
+"@reactive-dot/vue": patch
+---
+
+Fixed a bug that caused an error when retrieving accounts without specifying a chain ID.
diff --git a/packages/vue/src/composables/use-accounts.ts b/packages/vue/src/composables/use-accounts.ts
@@ -18,12 +18,14 @@ export function useAccounts(options?: ChainComposableOptions) {
 }
 
 function useAccountsPromise(options?: ChainComposableOptions) {
-  const chainId = internal_useChainId(options);
+  const chainId = internal_useChainId({ ...options, optionalChainId: true });
   const connectedWalletsObservable = useConnectedWalletsObservable();
   const chainSpecPromise = useChainSpecPromise(options);
 
   return useLazyValue(
-    computed(() => ["accounts", chainId.value]),
+    computed(() =>
+      chainId.value === undefined ? ["accounts"] : ["accounts", chainId.value],
+    ),
     () => getAccounts(connectedWalletsObservable.value, chainSpecPromise.value),
   );
 }