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

Remove auto connect wallet in browser #559

Merged
merged 18 commits into from
Jan 31, 2024
Merged

Remove auto connect wallet in browser #559

merged 18 commits into from
Jan 31, 2024

Conversation

trungbach
Copy link
Contributor

What is purpose of the changes?

  • Remove auto connect when user refresh page in browser
  • Just init client when user connect wallet, not when page refresh to reduce popup connect wallet -> improve UX

@trungbach trungbach changed the title Task/odex 140 Remove auto connect wallet in browser Jan 23, 2024
@trungbach trungbach closed this Jan 25, 2024
@trungbach trungbach reopened this Jan 31, 2024
Copy link

cloudflare-workers-and-pages bot commented Jan 31, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 26ba109
Status: ✅  Deploy successful!
Preview URL: https://55c4fbbb.oraiswap-frontend.pages.dev
Branch Preview URL: https://develop-v3.oraiswap-frontend.pages.dev

View logs

@trungbach trungbach requested a review from binhtt01 January 31, 2024 08:13
@trungbach trungbach merged commit f05a717 into main Jan 31, 2024
2 checks passed
@quangdz1704 quangdz1704 deleted the task/ODEX-140 branch January 31, 2024 08:29
Copy link

sentry-io bot commented Feb 3, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ Error: provider is not ready TrustWeb3Provider.postMessage(<unknown module>) View Issue
  • ‼️ Error: provider is not ready <anonymous>(hooks/useMetamask) View Issue
  • ‼️ Error: provider is not ready TrustWeb3Provider.postMessage(bridge) View Issue

Did you find this useful? React with a 👍 or 👎

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.

3 participants