Skip to content

Deploying public pool on stage #130

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.

Sign up for GitHub

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

Jump to bottom
Merged
lastperson merged 24 commits into from
Dec 2, 2025
Merged

Deploying public pool on stage #130

lastperson merged 24 commits into from
Dec 2, 2025

Conversation

@viatrix
Copy link
Collaborator

@viatrix viatrix commented Nov 10, 2025

No description provided.

lastperson and others added 17 commits October 28, 2025 19:11
@viatrix viatrix requested a review from lastperson November 10, 2025 21:00
@lastperson
Update network.config.ts
03b50cd 
We rebalance to/from ERC4626Adapter, which in turn deposits/withdraws from the Public pool.
@viatrix
Copy link
Collaborator Author

viatrix commented Nov 11, 2025 •
edited
Loading

Transactions to execute for adding routes:
OP_MAINNET:

To allow missing routes execute the following transaction.
To: 0x34Df9cD75fA0b4d8fF300AeDf7591f79dFdd61C9
Function: setRoute
Params:
isAllowed: true
┌─────────┬──────────────────────────────────────────────┬─────────┬───────────┐
│ (index) │ pools                                        │ domains │ providers │
├─────────┼──────────────────────────────────────────────┼─────────┼───────────┤
│ 0       │ '0xd7B2FC36b3D967DE88f9A4e3C84b7dBEF73e7eF3' │ 3n      │ 1n        │
└─────────┴──────────────────────────────────────────────┴─────────┴───────────┘

ARBITRUM_ONE:

To allow missing routes execute the following transaction.
To: 0x34Df9cD75fA0b4d8fF300AeDf7591f79dFdd61C9
Function: setRoute
Params:
isAllowed: true
┌─────────┬──────────────────────────────────────────────┬─────────┬───────────┐
│ (index) │ pools                                        │ domains │ providers │
├─────────┼──────────────────────────────────────────────┼─────────┼───────────┤
│ 0       │ '0xd7B2FC36b3D967DE88f9A4e3C84b7dBEF73e7eF3' │ 3n      │ 0n        │
└─────────┴──────────────────────────────────────────────┴─────────┴───────────┘

BASE:

To allow missing routes execute the following transaction.
To: 0x34Df9cD75fA0b4d8fF300AeDf7591f79dFdd61C9
Function: setRoute
Params:
isAllowed: true
┌─────────┬──────────────────────────────────────────────┬─────────┬───────────┐
│ (index) │ pools                                        │ domains │ providers │
├─────────┼──────────────────────────────────────────────┼─────────┼───────────┤
│ 0       │ '0xd7B2FC36b3D967DE88f9A4e3C84b7dBEF73e7eF3' │ 3n      │ 1n        │
└─────────┴──────────────────────────────────────────────┴─────────┴───────────┘

@viatrix viatrix requested a review from MakMuftic November 11, 2025 06:45
@viatrix viatrix marked this pull request as ready for review November 11, 2025 06:45
Base automatically changed from chore-public-pool-scripts to feat-public-pool November 20, 2025 16:25
@lastperson
Fix chain id in tests
ac40032
@lastperson
Audit fixes 4 (#135)
6394b9f 
* Apply audit fixes 1

* Add various fixes

* Remove console.log from tests

* Merge main and fix tests

* Make all tests run

* Last part of fixes

* Make public pool maxWithdraw and maxRedeem compliant with the EIP4626

* Fix lint

* Improve ERC4626Adapter withdraw profit safety assertion
@lastperson
Merge branch 'feat-public-pool' into chore-deploy-public-pool-stage
fa934c0
Base automatically changed from feat-public-pool to main December 2, 2025 10:59
lastperson
lastperson approved these changes Dec 2, 2025
View reviewed changes
Copy link
Collaborator

@lastperson lastperson left a comment

Choose a reason for hiding this comment

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

Merging for the logs, even though the public pool stage will be redeployed.

@lastperson lastperson merged commit 18cc519 into main Dec 2, 2025
3 checks passed
@lastperson lastperson deleted the chore-deploy-public-pool-stage branch December 2, 2025 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lastperson lastperson lastperson approved these changes

@MakMuftic MakMuftic Awaiting requested review from MakMuftic

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@viatrix @lastperson