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

feat: cvx and cvxcrv divest #1541

Merged
merged 4 commits into from
May 30, 2024
Merged

feat: cvx and cvxcrv divest #1541

merged 4 commits into from
May 30, 2024

Conversation

sajanrajdev
Copy link
Collaborator

Tackles #1536

Run trops prereq:

brownie run scripts/issue/1536/divest_voter_cvx.py transfer_fees_to_voter

Run main script:

brownie run scripts/issue/1536/divest_voter_cvx.py

@sajanrajdev
Copy link
Collaborator Author

@petrovska-petro good catch. Added the script for approving contract access and ran both within the same fork and no more reverts where found. In addition, I added a transfer of all other relevant fee assets to the voter from trops as per your recommendation. Here is the Trop snapshot:

snapshot result for 0x042B32Ac6b453485e357938bdC38e0340d4b9276:
                       balance_before         balance_after            balance_delta
symbol
CVX            317.253171502442701791  0.000000000000000000  -317.253171502442701791
cvxCRV         426.417428561462736675  0.000000000000000000  -426.417428561462736675
bveCVX          39.783485955818800628  0.000000000000000000   -39.783485955818800628
bcvxCRV          0.003175215823265107  0.000000000000000000    -0.003175215823265107
bbveCVX-CVX-f    3.294842883733444845  0.000000000000000000    -3.294842883733444845
bveCVX-CVX-f     1.487414004674213631  0.000000000000000000    -1.487414004674213631

Should do some nice clean-up.

@sajanrajdev sajanrajdev merged commit 705541a into main May 30, 2024
3 checks passed
@sajanrajdev sajanrajdev deleted the issue-1536 branch May 30, 2024 13:48
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