Skip to content
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

✨ Onboard AUSD Aave v3 Avalanche #530

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

MartinGbz
Copy link
Contributor

@MartinGbz MartinGbz commented Nov 25, 2024

I've added a transfer of the seed amount from the Collector to the Executor, because an error has been done in the transfer of the seed funds.

@MartinGbz MartinGbz force-pushed the feat/add-ausd-aave-v3-avalanche branch from f64e227 to b21f828 Compare November 25, 2024 13:41
@MartinGbz MartinGbz marked this pull request as ready for review November 27, 2024 11:06
@sakulstra
Copy link
Contributor

The risk assessment by us is still pending on this one and probably needs a capo feed.
Will get back to you once i got news on the topic.

@MartinGbz
Copy link
Contributor Author

@sakulstra Okay np, thanks for the info

Just have a question, I got a weird error on the test_defaultProposalExecution test.
[FAIL: revert: stdStorage find(StdStorage): Failed to write value.] => it seems to be an error with deal() from foundry

I didn't have this error before, and I still don't have this error in other AIP PRs.
I tried forge install, forge update, forge build --force, forge clean... etc, but still have the issue.

Any idea?

@sakulstra
Copy link
Contributor

@MartinGbz didn't yet check but probably AUSD does some fancy things on _balances storage.
Foundry deal essentially checks if it's a uint and if not reverts.
We maintain a list of tokens here: https://github.com/bgd-labs/aave-helpers/blob/master/src/CommonTestBase.sol#L45 that cannot be dealt.

@MartinGbz
Copy link
Contributor Author

@sakulstra Thanks for this, now it works like a charm. The lib is updated, test are passing, the PR is ready for a review.

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