search

yearn / ape-tax

https://ape.tax
46 stars 34 forks source link

chore: Add the 'useNetwork' hook of wagmi #224

Closed Itzabelli closed 1 year ago

Itzabelli commented 1 year ago

Description

The purpose of this PR is to follow up recommendations made by the team. Where it is suggested to use the useNetwork hook available in wagmi instead the local hook.

Related Issue

Requested changes: https://github.com/saltyfacu/ape-tax/pull/214#pullrequestreview-1542980817

Motivation and Context

The motivation to apply this change on top of reducing the code, is to seize the utilities available in wagmi.

How Has This Been Tested?

After making the changes I ran yarn dev and confirmed that everything appeared as expected.

Resources:

N/A

vercel[bot] commented 1 year ago

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

Name Status Preview Comments Updated (UTC)
ape-tax ❌ Failed (Inspect) Jul 27, 2023 9:47am