Skip to content

Fluid-wstETH-Mainnet#307

Merged
mkflow27 merged 6 commits into
mainfrom
Fluid-wstETH-Mainnet
Mar 20, 2025
Merged

Fluid-wstETH-Mainnet#307
mkflow27 merged 6 commits into
mainfrom
Fluid-wstETH-Mainnet

Conversation

@Zen-Maxi
Copy link
Copy Markdown
Contributor

Adding f-wstETH for Fluid. Lido intends to prioritize this over Lido Prime Aave market for wstETH.

Pending buffer seed from Fluid team.

Adding f-wstETH for Fluid. Lido intends to prioritize this over Lido Prime Aave market for wstETH.

Pending buffer seed from Fluid team.
@Zen-Maxi Zen-Maxi requested a review from zekraken-bot March 18, 2025 15:51
@Zen-Maxi Zen-Maxi requested a review from mkflow27 March 18, 2025 16:00
@Zen-Maxi
Copy link
Copy Markdown
Contributor Author

@mkflow27 if you can give us a final review, we added this information in for the Fluid-wstETH vault. Would be great to be green lit on March 19th. Will not merge until buffer is seeded and share txn link here.

Comment thread erc4626/registry.json
"summary": "safe",
"review": "/FluidVaults.md",
"warnings": [],
"canUseBufferForSwaps": true,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Quoting John

Hey. We want to be strict in the process of deciding if an ERC4626 is valid for swaps. Mainly because we cause 1inch, etc issues if we give them false info. During the review process canUseBufferForSwaps will only be set to true when both are true: - passes the ERC4626 fork tests - has an initialised Balancer V3 Buffer.

So, go ahead and init the buffer. Afterwards I will merge.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@mkflow27 mkflow27 merged commit 7d53067 into main Mar 20, 2025
@mkflow27 mkflow27 deleted the Fluid-wstETH-Mainnet branch March 20, 2025 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants