-
Notifications
You must be signed in to change notification settings - Fork 99
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
Re-enable buybacks #390
Re-enable buybacks #390
Conversation
Instead of 1->99 that created a lot of slippage
v2 proposal has already been executed onchain
|
||
const txReceipt = await tx.wait(); | ||
const { logs: rawLogs } = txReceipt; | ||
const noFeeFeiTribeLBPAddress = `0x${rawLogs[rawLogs.length - 1].topics[1].slice(-40)}`; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this code will never die ;)
|
||
export const validate: ValidateUpgradeFunc = async (addresses, oldContracts, contracts) => { | ||
// pcvEquityMinter should target the new LBPSwapper | ||
expect(await contracts.pcvEquityMinter.target()).to.be.equal(addresses.noFeeFeiTribeLBPSwapper); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
prefer to pull this variable out of the address object for readability
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like to use contracts.*
and addresses.*
for readability actually
|
||
// No tokens should remain anywhere on our contracts | ||
expect(await contracts.fei.balanceOf(addresses.feiTribeLBPSwapper)).to.be.equal('0'); | ||
expect(await contracts.tribe.balanceOf(addresses.feiTribeLBPSwapper)).to.be.equal('0'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same comment as above. prefer to pull out of the address object for readability
9ef5a9c
to
9144dd1
Compare
Re-enable buybacks
Last week, we had to use the Guardian to pause the buybacks. This proposal re-enables buybacks.
Also updates the LBPSwapper to shift weights from 95% -> 5%, instead of 99% -> 1%.
Proposal calldata :