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

chore: upgrade libs wagmi, rainbowkit, viem #3

Merged
merged 2 commits into from
Mar 15, 2024
Merged

Conversation

0xtiti
Copy link
Contributor

@0xtiti 0xtiti commented Mar 14, 2024

Upgrade libraries with breaking changes

Description:

  • New API that includes Wagmi's createConfig and replaces getDefaultWallets and connectorsForWallets
  • Instantiating wallet connectors and passing projectId and chains is no longer required.

Checklist before requesting a review

  • I have conducted a self-review of my code.
  • If it is a core feature, I have included comprehensive tests.
  • Do we need to implement analytics?
  • Will this be part of a product update? If yes, please write one sentence about this update.

package.json Outdated Show resolved Hide resolved
@turtlemoji
Copy link

check semantic pr name

@0xtiti 0xtiti changed the title Libs upgrade: wagmi, rainbowkit, viem chore: upgrade libs wagmi, rainbowkit, viem Mar 14, 2024
@0xtiti 0xtiti force-pushed the chore/update-libs branch from 50c22c1 to 0d7a34f Compare March 14, 2024 23:42
src/main.tsx Outdated Show resolved Hide resolved
0xArdy
0xArdy previously approved these changes Mar 15, 2024
Copy link
Member

@0xArdy 0xArdy left a comment

Choose a reason for hiding this comment

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

great work! 🚀
now after resolving the conflicts you can Squash and merge

chore: wagmi, rainbowkit, viem updates

tests: rainbokit 2.x breaking changes updates

fix: injected wallet for empty projectId logic

fix: wallets type error - projectid not required rainbowWallet nor walletConnect

fix: caret removed

fix: group imports

chore: update viem and wagmi to latest version (#1)

fix: yarn libs error
@0xtiti 0xtiti force-pushed the chore/update-libs branch from 020379f to 62afcc5 Compare March 15, 2024 14:26
@0xArdy 0xArdy merged commit ceb8995 into dev Mar 15, 2024
4 checks passed
@0xArdy 0xArdy deleted the chore/update-libs branch March 15, 2024 14:48
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.

3 participants