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: migrate to wagmi #205

Merged
merged 10 commits into from
Jun 5, 2023
Merged

feat: migrate to wagmi #205

merged 10 commits into from
Jun 5, 2023

Conversation

Majorfi
Copy link
Collaborator

@Majorfi Majorfi commented May 30, 2023

Migrate from ethers/ethcall/web3-react to wagmi/viem

Testing:

  • common.isApprovedERC20
  • common.approvedERC20Amount
  • common.approveERC20
  • common.deposit
  • common.depositETH
  • common.depositViaPartner
  • common.withdrawETH
  • common.withdrawShares
  • common.migrateShares
  • vaults.depositAndStake
  • vaults.stake
  • vaults.unstake
  • vaults.claim
  • vaults.veCRVzap
  • vaults.createNewVaultsAndStrategies
  • vaults.gasOfCreateNewVaultsAndStrategies
  • solvers.cowswap
  • solvers.wido
  • solvers.portals
  • yCRV.zapCRV
  • yBal.zapBal
  • veYFI.lockVeYFI
  • veYFI.increaseVeYFILockAmount
  • veYFI.extendVeYFILockTime
  • veYFI.getVeYFIWithdrawPenalty
  • veYFI.withdrawUnlockedVeYFI
  • veYFI.withdrawLockedVeYFI
  • yBribes.claimRewardV3
  • yBribes.addReward

@Majorfi Majorfi added the enhancement New feature or request label May 30, 2023
@vercel
Copy link

vercel bot commented May 30, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
yearnfi ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 5, 2023 2:53pm

karelianpie
karelianpie previously approved these changes Jun 1, 2023
Copy link
Member

@karelianpie karelianpie left a comment

Choose a reason for hiding this comment

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

LGTM

apps/common/components/BalanceReminderPopover.tsx Outdated Show resolved Hide resolved
apps/common/utils/actions/approveToken.tsx Outdated Show resolved Hide resolved
apps/common/utils/actions/deposit.tsx Outdated Show resolved Hide resolved
apps/common/utils/actions/withdraw.tsx Outdated Show resolved Hide resolved
apps/common/utils/toWagmiProvider.tsx Show resolved Hide resolved
pages/api/getBatchBalances.tsx Show resolved Hide resolved
karelianpie
karelianpie previously approved these changes Jun 2, 2023
Majorfi and others added 3 commits June 3, 2023 10:47
* feat: migrate common

* feat: working on vaults hooks

* fix: undo test void

* fix: remove useless lint

* fix: remove useless code

* fix: undo change for upcoming pr

* fix: undo change for upcoming pr

* fix: undo change for upcoming pr

* fix: reduce complexity and change import constants

* Update apps/common/utils/toWagmiProvider.tsx

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>

* Update apps/common/utils/toWagmiProvider.tsx

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>

* Update apps/vaults/contexts/useSolver.tsx

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>

* feat: migrate utils

* fix: remove comment

* chore: move to ts

* fix: tsconfig autopopulate

* Update apps/vaults/hooks/useSolverCowswap.ts

* fix: some assert after review

* Update apps/common/utils/toWagmiProvider.tsx

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>

* Update apps/common/utils/toWagmiProvider.tsx

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>

* Update apps/vaults/contexts/useSolver.tsx

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>

* Update apps/vaults/hooks/useSolverCowswap.ts

* fix: some assert after review

* feat: migrate utils

* fix: rebase err

* Update tsconfig.json

* Update tsconfig.json

* Update apps/common/components/BalanceReminderPopover.tsx

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>

* feat: working on vaults hooks

* fix: reduce complexity and change import constants

* fix: some assert after review

* feat: sentry catch

* fix: ensure hex is hex

* fix: toChainID API

* fix: use ternary

* fix: toWagmiAddress

* Feat/wagmi vaults (#208)

* feat: wagmi in vaults

* lint: all \t to spaces for const, let and functions (#209)

* fix: lint

* fix: lint

* Feat/wagmi veyfi (#210)

* feat: migrate to wagmi

* Update pages/vaults/factory.tsx

* Feat/wagmi yBribe (#211)

* feat: migrate to wagmi

* Feat/wagmi ybal (#214)

* feat: init yBal

* fix: lint

* Update apps/common/components/AppHeader.tsx

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>

* fix: remove unused test prop

* feat: refactor useCurrentApp

* fix: reduce lines

* fix: extract HeadLabel

* fix: add more strict checks

* fix: reduce complexity

* fix: extract Row

* fix: invalid code

* fix: possible undef

* feat: update lib and var names

* feat: update options from/to

* wip: yBal no yCRV

* fix: remove veBal

* feat: cleaning imports and options

* feat: bal zap integration WIP

* fix: onCatchError

* test: yBal actions#zaap

* test: simulateZapForMinOut

* refactor: Expect zap to be called with BigNumber

* feat: update lib and var names

* feat: new contract version

* wip: ugly

* fix: Ethers mock

* feat: Mock providers

* fix: invalid BigNumber value

* feat: update tests

* feat: always test

* fix: switch env to node for tests

* fix: remove some logs

* feat: wip yBal

* feat: wip yBal

* feat: ylock ui

* feat: useFetch for fetching stybal vault (#202)

Co-authored-by: Major <90963895+Majorfi@users.noreply.github.com>

* fix: conflicts and yCRV clean

* feat: smol refactor

* feat: yBal pull

* feat: yBal migration

* fix: spacings

* fix: slippage

* fix: remove extra props for arrow down

* fix: add sentry to handleSettle

* fix: tabs

* Update apps/ycrv/utils/actions.ts

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>

* fix: set const

* fix: set const

* feat: update solvers (#217)

* feat: update solvers

* fix: dynamic imports

* fix: some lint issues

* fix: removing wdyu

* fix: remove log

* fix: enable

* fix: endpoint to path

* Feat/wagmi address (#219)

* feat: remove all wagmiAddress

* fix: linting

* fix: typing issues

* fix: remove useless assert and add namecheck

* fix: remove duplicate import

* Update apps/vaults/contexts/useStakingRewards.tsx

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>

* Feat/wagmi-bribesfix (#220)

* feat: remove v3 declaration

* fix: transform strings to addresses

* fix: bribe results

* fix: revert ui changes. Later

* fix: some bit of UI

* fix: responsiveness

* Update apps/common/schemas/curveSchemas.ts

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>

* fix: transform toAddress easy mode

* feat/Refactor tx handlers (#222)

* feat: wagmi handleTX init

* fix: remove some level of complexity

* feat: migrate the common actions

* feat: suggestion to make bigInt |undefined to be sure to check stuff

* fix: remove stupid suggestion

* feat: migrate actions

* fix: Types

* fix: Cannot set bigint to type undefined

* Update apps/common/utils/toWagmiProvider.tsx

* fix: remove useless toString() (#223)

---------

Co-authored-by: Karelian Pie <devkarelianpie@gmail.com>

---------

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>
Co-authored-by: Karelian Pie <devkarelianpie@gmail.com>

---------

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>
Co-authored-by: Karelian Pie <devkarelianpie@gmail.com>

---------

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>
Co-authored-by: Karelian Pie <devkarelianpie@gmail.com>

---------

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>
Co-authored-by: Karelian Pie <devkarelianpie@gmail.com>

---------

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>
Co-authored-by: Karelian Pie <devkarelianpie@gmail.com>

---------

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>
Co-authored-by: Karelian Pie <devkarelianpie@gmail.com>

---------

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>
Co-authored-by: Karelian Pie <devkarelianpie@gmail.com>

---------

Co-authored-by: Chef Karelian <78794805+karelianpie@users.noreply.github.com>
Co-authored-by: Karelian Pie <devkarelianpie@gmail.com>
@Majorfi Majorfi changed the title feat: migrate common feat: migrate to wagmi Jun 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants