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

[aavegotchi-agip] Update Aavegotchi Strategy #1596

Merged
merged 7 commits into from
Oct 1, 2024

Conversation

cinnabarhorse
Copy link
Contributor

Fixes # .

Changes proposed in this pull request:

  • Add new wearables to fix broken strategy

@cinnabarhorse cinnabarhorse changed the title Aavegotchi Update Aavegotchi Strategy Sep 30, 2024
@ChaituVR
Copy link
Member

@cinnabarhorse conflicts

@cinnabarhorse
Copy link
Contributor Author

Thanks, fixed! @ChaituVR

@ChaituVR
Copy link
Member

@cinnabarhorse Looks like there is an active proposal https://snapshot.org/#/aavegotchi.eth/proposal/0x2a4a3871f300701c734d4d3b6dc139dc161d1f7c7c5c7eab974158d85b393e46 if this can effect voting power, then some users may see wrong voting power due to our cache systems

@ChaituVR ChaituVR changed the title Update Aavegotchi Strategy [aavegotchi-agip] Update Aavegotchi Strategy Sep 30, 2024
@cinnabarhorse
Copy link
Contributor Author

@cinnabarhorse Looks like there is an active proposal https://snapshot.org/#/aavegotchi.eth/proposal/0x2a4a3871f300701c734d4d3b6dc139dc161d1f7c7c5c7eab974158d85b393e46 if this can effect voting power, then some users may see wrong voting power due to our cache systems

The current proposal is not fully reflecting everyone's accurate voting power due to a bug that this PR fixes. Can we merge it and I will have our community re-vote, instead of re-submitting the prop.

If the issue still isn't fixed then I will just have the poster re-submit the prop.

@ChaituVR ChaituVR merged commit 461a692 into snapshot-labs:master Oct 1, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants