Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use accountsChanged event for OKX #743

Merged
merged 3 commits into from
Oct 2, 2024
Merged

Use accountsChanged event for OKX #743

merged 3 commits into from
Oct 2, 2024

Conversation

kkosiorowska
Copy link
Contributor

@kkosiorowska kkosiorowska commented Sep 27, 2024

The previous event emitted a disconnect right at the start of the app. Let's use the event accountsChanged which is described in the documentation and behaves correctly.

More info: https://www.okx.com/web3/build/docs/sdks/chains/bitcoin/provider#accountschanged

Screen.Recording.2024-09-27.at.13.18.05.mov

The previous event emitted a disconnect right at the start of the app. Let's use the event `accountsChanged` which is described in the documentation and behaves correctly.

More info: https://www.okx.com/web3/build/docs/sdks/chains/bitcoin/provider#accountschanged
@kkosiorowska kkosiorowska self-assigned this Sep 27, 2024
Copy link

netlify bot commented Sep 27, 2024

Deploy Preview for acre-dapp ready!

Name Link
🔨 Latest commit 7201da4
🔍 Latest deploy log https://app.netlify.com/sites/acre-dapp/deploys/66fce611c08c5e0008fcc557
😎 Deploy Preview https://deploy-preview-743--acre-dapp.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Sep 27, 2024

Deploy Preview for acre-dapp-testnet ready!

Name Link
🔨 Latest commit 7201da4
🔍 Latest deploy log https://app.netlify.com/sites/acre-dapp-testnet/deploys/66fce611bf5d30000844c21a
😎 Deploy Preview https://deploy-preview-743--acre-dapp-testnet.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@r-czajkowski r-czajkowski enabled auto-merge October 2, 2024 06:20
@r-czajkowski r-czajkowski merged commit 7e184bf into main Oct 2, 2024
26 of 28 checks passed
@r-czajkowski r-czajkowski deleted the okx-event-fix branch October 2, 2024 06:22
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