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

WalletConnect + Wagmi + Viem Migration #669

Closed
3 tasks done
jaybuidl opened this issue Apr 5, 2023 · 5 comments · Fixed by #670
Closed
3 tasks done

WalletConnect + Wagmi + Viem Migration #669

jaybuidl opened this issue Apr 5, 2023 · 5 comments · Fixed by #670
Assignees
@alcercu
Labels
Package: Web Court web frontend Priority: High Type: Epic 🐉 A large user story that can be broken down into a number of smaller stories.
Milestone
testnet-1

Comments

@jaybuidl
Copy link
Member

jaybuidl commented Apr 5, 2023 •
edited
Loading

WalletConnect
Wagmi
Viem
@jaybuidl jaybuidl added Type: Epic 🐉 A large user story that can be broken down into a number of smaller stories. Package: Web Court web frontend Priority: High labels Apr 5, 2023
@jaybuidl jaybuidl changed the title WalletConnect + Wagmi epic WalletConnect + Wagmi Migration Apr 5, 2023
@Arun9650
Copy link
Contributor

Arun9650 commented Apr 7, 2023

hi like to work on these issues

@Arun9650
Copy link
Contributor

Arun9650 commented Apr 9, 2023 •
edited
Loading

i have done the integration should i raise a pr? #320

@jaybuidl
Copy link
Member Author

Hey @Arun9650 thanks for the contribution. Actually @alcercu has a PR in progress for this already #670.
We'll see if we can cherry pick some useful changes from your PR 🙌.

@Arun9650
Copy link
Contributor

Ohh that's great, I would like to contribute more in this project, it will be a pleasure to me if u redirect issues to me.

This was referenced Apr 21, 2023
Closed
Closed
@jaybuidl jaybuidl changed the title WalletConnect + Wagmi Migration WalletConnect + Wagmi + Viem Migration Jun 7, 2023
@jaybuidl
Copy link
Member Author

jaybuidl commented Jun 7, 2023

UPDATE: this issue has been been expanded to include the migration to Viem.sh which was originally deemed unnecessary. But since then Wagmi has totally dropped ethers and adopted viem for its first stable release and future ones going forward.

https://viem.sh/docs/ethers-migration.html

@jaybuidl jaybuidl added this to the alpha-1 milestone Jun 7, 2023
@jaybuidl jaybuidl linked a pull request Jun 7, 2023 that will close this issue
Refactor(web): migration to wagmi and viem #670
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alcercu alcercu

Labels
Package: Web Court web frontend Priority: High Type: Epic 🐉 A large user story that can be broken down into a number of smaller stories.
Projects
None yet
Milestone
testnet-1
Development

Successfully merging a pull request may close this issue.

Refactor(web): migration to wagmi and viem kleros/kleros-v2
3 participants
@jaybuidl @Arun9650 @alcercu