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

Add More Rpc Urls to Support Multiple Providers #1471

Merged
merged 5 commits into from
Aug 1, 2023
Merged

Add More Rpc Urls to Support Multiple Providers #1471

merged 5 commits into from
Aug 1, 2023

Conversation

AtelyPham
Copy link
Member

Summary of changes

  • Add more RPC URLs to support multiple providers
  • Fix the network storage by moving the networkSwitched event to be emitted outside of the if condition
  • Fix default Polygon Mumbai RPC URL from wagmi/references library is inactive -> Override the an active RPC
  • Add multiple provider support for getViemClient function.

Proposed area of change

Put an x in the boxes that apply.

  • apps/bridge-dapp
  • apps/hubble-stats
  • apps/stats-dapp
  • apps/webbsite
  • apps/faucet
  • apps/tangle-website
  • libs/webb-ui-components

@AtelyPham AtelyPham added the needs review 👓 Pull request needs a review label Jul 31, 2023
@AtelyPham AtelyPham requested a review from drewstone July 31, 2023 08:19
@AtelyPham AtelyPham self-assigned this Jul 31, 2023
@AtelyPham AtelyPham added the bridge Issues specific to the bridge label Jul 31, 2023
drewstone
drewstone approved these changes Aug 1, 2023
View reviewed changes
Copy link
Contributor

@drewstone drewstone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

libs/dapp-config/src/chains/evm/index.tsx Outdated Show resolved Hide resolved
@vutuanlinh2k2
Copy link
Contributor

LGTM too!

@AtelyPham AtelyPham merged commit 27dcf65 into develop Aug 1, 2023
6 checks passed
@AtelyPham AtelyPham deleted the trung-tin/multiple-providers branch August 1, 2023 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vutuanlinh2k2 vutuanlinh2k2 vutuanlinh2k2 approved these changes

@drewstone drewstone drewstone approved these changes

@devpavan04 devpavan04 Awaiting requested review from devpavan04 devpavan04 is a code owner

Assignees

@AtelyPham AtelyPham

Labels
bridge Issues specific to the bridge needs review 👓 Pull request needs a review
Projects
Webb Universe
Status: Completed ✅
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AtelyPham @vutuanlinh2k2 @drewstone