Skip to content

Discontinue Tribe Incentives#868

Merged
thomas-waite merged 36 commits intodevelopfrom
end-tribe-incentives
Jun 3, 2022
Merged

Discontinue Tribe Incentives#868
thomas-waite merged 36 commits intodevelopfrom
end-tribe-incentives

Conversation

@thomas-waite
Copy link
Contributor

@thomas-waite thomas-waite commented Jun 1, 2022

Discontinue TRIBE incentives

Enact the passed TribalCouncil proposal to discontinue Tribe incentives.

Specifically this:

  • Unlocks all pools to allow principle to be withdrawn now that rewards have ended
  • Sets all pool APs to zero, apart from pool 3 which has it's AP set 1 - to avoid potential divide by zero errors
  • Sets the tribe block reward to 1e6, effectively 0. This globally discontinues rewards
  • Removes the CREAM deposit from the CR

Items for later

  1. Harvest the staking token wrappers
  2. Calculate how much TRIBE is owed out to users
  3. Withdraw excess TRIBE from the TribalChief back to Treasury

@thomas-waite thomas-waite requested a review from a team as a code owner June 1, 2022 17:09
@thomas-waite thomas-waite self-assigned this Jun 1, 2022
@thomas-waite thomas-waite changed the title End Tribe Incentives Discontinue Tribe Incentives Jun 1, 2022
ElliotFriedman
ElliotFriedman previously approved these changes Jun 1, 2022
Copy link
Contributor

@ElliotFriedman ElliotFriedman left a comment

Choose a reason for hiding this comment

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

LGTM!

@eswak
Copy link
Contributor

eswak commented Jun 1, 2022

@thomas-waite
Copy link
Contributor Author

thomas-waite commented Jun 1, 2022

Update of https://github.com/fei-protocol/fei-protocol-core/blob/develop/protocol-configuration/tribalchief.ts not needed ? I'm surprised

Simplest solution is to just tell the TribalChief that new block rewards are 0. Alternative approach would have been to set the APs of all pools to 0

@thomas-waite thomas-waite force-pushed the end-tribe-incentives branch from cac88e9 to bd07d8d Compare June 2, 2022 15:13
@thomas-waite thomas-waite force-pushed the end-tribe-incentives branch from 32b646e to 333c21e Compare June 2, 2022 16:54
@thomas-waite thomas-waite force-pushed the end-tribe-incentives branch from df9feff to 86dceff Compare June 3, 2022 15:56
@thomas-waite thomas-waite force-pushed the end-tribe-incentives branch from 5af053c to 20309b3 Compare June 3, 2022 16:01
@thomas-waite thomas-waite merged commit c7fb0f6 into develop Jun 3, 2022
@thomas-waite thomas-waite deleted the end-tribe-incentives branch June 3, 2022 17:36

const delegatorReportedSpeed = await contracts.rariRewardsDistributorDelegator.compSupplySpeeds(d3Ctoken);
expect(delegatorReportedSpeed).to.equal(0);
};
Copy link
Contributor

Choose a reason for hiding this comment

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

you should verify here that all AutoRewardsDistributors are set to 0 alloc points as well.

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.

4 participants

Comments