Skip to content

feat: wagmi and viem v1 #448

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
Open
wants to merge 87 commits into
base: dev
Choose a base branch
from
Open

feat: wagmi and viem v1 #448

wants to merge 87 commits into from

Conversation

brucedonovan
Copy link
Contributor

No description provided.

@netlify
Copy link

netlify bot commented Sep 14, 2023 •
edited
Loading

✅ Deploy Preview for ironclad-parent ready!

Name Link
🔨 Latest commit 230d31b
🔍 Latest deploy log https://app.netlify.com/sites/ironclad-parent/deploys/65143de98152e3000807960f
😎 Deploy Preview https://deploy-preview-448--ironclad-parent.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

marcomariscal and others added 26 commits September 14, 2023 09:41
@marcomariscal
package: update viem
5925d72
@marcomariscal
fix: update to solely use viem
630a836
@marcomariscal
feat: update to use viem types
9f925c7
@marcomariscal
fix: remove unused ethers types
08f556a
@marcomariscal
feat: use prepare contract write type
56884bc
@marcomariscal
fix: update to use viem
6e49155
@marcomariscal
chore: remove unused
b357810
@marcomariscal
fix: use prepare contract write type
c629dc1
@marcomariscal
fix: update to use viem
add1b34
@marcomariscal
feat: update to use viem
595f69e
@marcomariscal
fix: update to use viem
389b69b
@marcomariscal
chore: delete legacy/unused buy nft component
363f199
@marcomariscal
fix: update to use viem
5618450
@marcomariscal
fix: update to use viem
5023c2d
@marcomariscal
fix: use viem
12792b8
@brucedonovan
ethersAdapter
b782382
@marcomariscal
feat: use viem
a28390c
@marcomariscal
feat: use viem
8207d57
@marcomariscal
chore: remove unused
f99a937
@marcomariscal
feat: remove ethers
a62f2db
@marcomariscal
feat: remove ethers
cdeae88
@marcomariscal
feat: use transaction request base type from wagmi/viem
7fe19d3
@marcomariscal
Merge branch 'feat/newWagmiViem' into feat/newWagmiViem-yield
7551bf8 
merging latest wagmi/viem
@marcomariscal
feat: trying to update to use solely viem
bf53e67
@marcomariscal
feat: update to use viem
fe7807f
@marcomariscal
feat: update to only use viem
cb713d4
@marcomariscal
Copy link
Contributor

marcomariscal commented Sep 21, 2023 •
edited by brucedonovan
Loading

Further testing needed for each component.

Is manual testing completed by widget:

  • 4626
  • dsr
  • frax staking
  • hop bridging
  • lido deposit
  • liquity borrow (not integrated into backend yet)
  • liquity close (not integrated into backend yet)
  • buy nft
  • nft asset
  • nft asset list
  • nft collection
  • rocket pool deposit
  • rocket pool withdraw
  • transfer
  • tx replay
  • uniswap
  • weth (not on BE?)
  • yield borrow
  • yield close
  • yield lend
  • yield close
  • zksync deposit
  • zksync withdraw

@marcomariscal marcomariscal changed the title Feat/new wagmi viem feat: wagmi and viem v1 Sep 21, 2023
@brucedonovan brucedonovan marked this pull request as ready for review September 25, 2023 11:00
marcomariscal
marcomariscal approved these changes Sep 27, 2023
View reviewed changes
Copy link
Contributor

@marcomariscal marcomariscal left a comment

Choose a reason for hiding this comment

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

approving, knowing that a few of the widgets need more stability (separate context from this pr)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marcomariscal marcomariscal marcomariscal approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@brucedonovan @marcomariscal