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

Uses Transaction Fees to pay rewards instead of minting new tokens #235

Merged
merged 3 commits into from
Aug 25, 2021

Conversation

joshuahannan
Copy link
Member

Closes #233

Adds a change to use fees from the FlowFees smart contract to pay rewards instead of minting new tokens
The rewards payments take from the fees vault first, and once that has run out, it will start minting new tokens.
If there are still fees left after all rewards have been paid, those are burned.

Adds an event documenting the details of the reward payments

contracts/FlowIDTableStaking.cdc Outdated Show resolved Hide resolved
contracts/FlowIDTableStaking.cdc Outdated Show resolved Hide resolved
lib/go/contracts/contracts.go Outdated Show resolved Hide resolved
Copy link
Member

@jordanschalm jordanschalm left a comment

Choose a reason for hiding this comment

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

LGTM

contracts/FlowIDTableStaking.cdc Show resolved Hide resolved
Copy link
Member

@jordanschalm jordanschalm left a comment

Choose a reason for hiding this comment

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

🎸

@joshuahannan joshuahannan merged commit 116ef28 into master Aug 25, 2021
@joshuahannan joshuahannan deleted the fee-allocation branch August 25, 2021 23:04
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.

Update Staking Contract to Use Transaction Fees for Staking Rewards
3 participants