-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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: Remove turkey banner and spread reduce #10840
base: develop
Are you sure you want to change the base?
chore: Remove turkey banner and spread reduce #10840
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
6 Skipped Deployments
|
may I check why do we need to update the spread use case? |
ec1977d
to
8e55da8
Compare
PR-Codex overview
This PR primarily focuses on optimizing the use of the
reduce
method across various components by replacing object spread syntax with direct assignments to improve performance and readability.Detailed summary
pools.ts
, changed return ofreduce
to useaccum.push
.StableV3Selector.tsx
, modifiedreduce
to directly assign values instead of spreading.IncreaseLiquidityV3.tsx
,V3FormView/index.tsx
, andFeeSelector.tsx
.maxAmounts
calculation inAddStableLiquidity/index.tsx
to useuseMemo
with direct assignments.useMultipleBannerConfig
to remove a banner entry.usePositionsByUserFarms
anduseV3BoostedLiquidityX
to use direct assignments inreduce
.