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

feat(website): Improve behavior on adding chains and tokens #14781

Merged
merged 2 commits into from
Sep 25, 2023

Conversation

Nakochi-crypto
Copy link
Contributor

This pull request improves these behaviors below on the docs.

  • Switch chains to Sepolia before adding Sepolia tokens
  • Fix a bug where the current chainId won't be changed

https://taiko.xyz/docs/guides/setup-your-wallet

Also, this removes ethereumRequest because it won't throw errors and the callers can't handle them.

@vercel
Copy link

vercel bot commented Sep 22, 2023

@Nakochi-crypto is attempting to deploy a commit to the taikoxyz Team on Vercel.

A member of the Team first needs to authorize it.

@d1onys1us
Copy link
Contributor

this is a great improvement, thanks @Nakochi-crypto!

@d1onys1us d1onys1us added this pull request to the merge queue Sep 25, 2023
Merged via the queue into taikoxyz:main with commit fa59e09 Sep 25, 2023
5 of 8 checks passed
@github-actions github-actions bot mentioned this pull request Sep 25, 2023
@Nakochi-crypto Nakochi-crypto deleted the add-chains-and-tokens branch September 26, 2023 08:53
@Nakochi-crypto
Copy link
Contributor Author

@d1onys1us Thanks! Can you give me a GitPoap?

@d1onys1us
Copy link
Contributor

@d1onys1us Thanks! Can you give me a GitPoap?

@gitpoap-bot please award @Nakochi-crypto

@gitpoap-bot
Copy link

gitpoap-bot bot commented Sep 26, 2023

Congrats, @Nakochi-crypto ! You've earned a GitPOAP for your contribution!

GitPOAP: 2023 Taiko Contributor:

GitPOAP: 2023 Taiko Contributor GitPOAP Badge

Head to gitpoap.io & connect your GitHub account to mint!

Learn more about GitPOAPs here.

KorbinianK pushed a commit that referenced this pull request Sep 28, 2023
Co-authored-by: dave | d1onys1us <13951458+d1onys1us@users.noreply.github.com>
KorbinianK pushed a commit that referenced this pull request Sep 28, 2023
Co-authored-by: dave | d1onys1us <13951458+d1onys1us@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants