Skip to content

Conversation

joshzhang5
Copy link
Collaborator

@joshzhang5 joshzhang5 commented Sep 16, 2025

This is breaking the useLogout hook for the react-native lib

Pull Request Checklist


PR-Codex overview

This PR focuses on enhancing the clearWalletConnectStore function to include an additional condition that checks for the availability of indexedDB, ensuring it only executes in environments where both window and indexedDB are defined.

Detailed summary

  • Updated the clearWalletConnectStore function's conditional check.
  • Added a condition to verify that window.indexedDB is defined.
  • Ensured that the function returns early if either window or indexedDB is unavailable.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

vercel bot commented Sep 16, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
aa-sdk-ui-demo Ready Ready Preview Comment Sep 16, 2025 6:50pm

@joshzhang5 joshzhang5 changed the title 09 15 fix don t try to clear walletconnect data for react native fix: don't try to clear walletconnect data for react native Sep 16, 2025
Copy link

github-actions bot commented Sep 16, 2025

🌿 Documentation Preview

Name Status Preview Updated (UTC)
Alchemy Docs ✅ Ready 🔗 Visit Preview Sep 16, 2025, 6:46 PM

@github-actions github-actions bot temporarily deployed to docs-preview September 16, 2025 18:38 Inactive
@joshzhang5 joshzhang5 merged commit 75a0f66 into main Sep 16, 2025
13 checks passed
@joshzhang5 joshzhang5 deleted the 09-15-fix_don_t_try_to_clear_walletconnect_data_for_react_native branch September 16, 2025 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants