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.

Sign up for GitHub

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

Jump to bottom

feat(bridge-ui): Simplify the wallet connection modal #509

Merged
merged 2 commits into from
Jan 9, 2025
Merged

feat(bridge-ui): Simplify the wallet connection modal #509

merged 2 commits into from
Jan 9, 2025

Conversation

alainncls
Copy link
Contributor

@alainncls alainncls commented Jan 8, 2025 •
edited
Loading

Fixes #514

  • Updates the web3 dependencies (wagmi, viem)
  • Switches from Web3Modal to Reown
  • Simplifies the connect wallet modal (showing only EOAs)

@alainncls alainncls self-assigned this Jan 8, 2025
@cla-assistant CLA Assistant
Copy link

cla-assistant bot commented Jan 8, 2025 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@cla-assistant CLA Assistant
Copy link

cla-assistant bot commented Jan 8, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@alainncls @thedarkjester
Fix copy on metadata
dea877a 
Co-authored-by: The Dark Jester <[email protected]>
Signed-off-by: Alain Nicolas <[email protected]>
@alainncls alainncls temporarily deployed to docker-build-and-e2e January 8, 2025 16:58 — with GitHub Actions Inactive
@alainncls alainncls merged commit 06294fd into main Jan 9, 2025
23 checks passed
@alainncls alainncls deleted the feat/simplify-connect-modal-bridge-ui branch January 9, 2025 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@VGau VGau VGau approved these changes

@thedarkjester thedarkjester thedarkjester approved these changes

@julien-marchand julien-marchand Awaiting requested review from julien-marchand

Assignees

@alainncls alainncls

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Simplify the Bridge Connect Wallet modal
3 participants
@alainncls @thedarkjester @VGau