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

fix: Enable network switcher menu for Coinbase Wallet #446

Merged
merged 1 commit into from
Jan 6, 2022

Conversation

brendanww
Copy link
Contributor

@brendanww brendanww commented Nov 18, 2021

Enables the 3085 network switcher menu for Coinbase Wallet (dapp browser in mobile app, walletlink via mobile device, and walletlink via Coinbase Wallet chromium extension). Also updates walletlink-connector to a version that supports 3085 requests.

Prior PR into canary at
#420

Equivalent PR into uniswap
Uniswap/interface#2753

@vercel
Copy link

vercel bot commented Nov 18, 2021

@brendanww is attempting to deploy a commit to the Team Sushi Team on Vercel.

A member of the Team first needs to authorize it.

@brendanww
Copy link
Contributor Author

@jiro-ono would be great to get some 👀 on this

@brendanww
Copy link
Contributor Author

@matthewlilley a polite nudge on checking out this PR

@matthewlilley
Copy link
Member

matthewlilley commented Jan 4, 2022

Hey @brendanww, would you mind opening a PR without translations included, and against canary. Ping me and I'll merge ASAP.

@brendanww
Copy link
Contributor Author

@matthewlilley updated! thank you 🙏

@matthewlilley
Copy link
Member

@brendanww fantastic! lgtm

@matthewlilley
Copy link
Member

@brendanww if you could include a PR for canary branch also that would be great, so it doesn't get lost.

@matthewlilley matthewlilley merged commit 2951dd5 into sushiswap:master Jan 6, 2022
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.

None yet

2 participants