-
Notifications
You must be signed in to change notification settings - Fork 32
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
Deposits on non-ETH gas token pools #1536
Conversation
WalkthroughThe changes primarily revolve around the enhancement of the Changes
Poem
TipsChat with CodeRabbit Bot (
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1536 +/- ##
=============================================
Coverage 51.23324% 51.23324%
=============================================
Files 362 362
Lines 24691 24691
Branches 283 283
=============================================
Hits 12650 12650
Misses 10813 10813
Partials 1228 1228
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Deploying with Cloudflare Pages
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but should we also update the token naming from ETH to WETH on Metis for the sake of consistency?
9ef5ccb
to
6caa989
Compare
6caa989
to
b0546cb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files selected for processing (1)
- packages/synapse-interface/pages/pool/poolManagement/Deposit.tsx (5 hunks)
Additional comments: 4
packages/synapse-interface/pages/pool/poolManagement/Deposit.tsx (4)
40-40: Ensure that the
zeroAddress
imported from theviem
module is the correct constant and is being used appropriately in the context of this file.209-215: The
pool
prop has been added to theSerializedDepositInput
component. Make sure that this prop is being passed correctly wherever this component is used.287-294: The
getBalanceToken
function (previouslycorrectToken
) has been modified. It now checks if thepool
does not include the native tokenWETH
before assigningETH
tobalanceToken
. This change seems logical and should work as expected.304-310: The
serializeToken
function has been updated to check if thebalanceToken
address matches the zero address before fetching the balance. This is a good practice as it prevents unnecessary calls to fetch the balance for non-ETH gas token pools.
null
c1314b8fa56b5de064b676049f1d0fa199626ea1: synapse-interface preview link
c7674e68aab0a5eccf7ee7498d1ff1c4e20c15ff: synapse-interface preview link
f13ff82d2e1e3ac294579892b65310356edbcddd: synapse-interface preview link
Summary by CodeRabbit