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

New Treasury reward module updates #168

Closed
wants to merge 4 commits into from
Closed

Conversation

drewstone
Copy link
Collaborator

@drewstone drewstone commented Jun 5, 2020

Description

This PR adds new, documented logic to the treasury reward module with more tests and compatibility at the time-travel commit. The functionality changes are:

  • Enabling splitting the existing treasury reward between potential recipients that are voted in and removed through governance votes (council or democracy).
  • Event firing for all changes to the parameters in the system
  • Minimum and maximum reward percentages for the Treasury and recipients to ensure that there are limits to the system's capabilities and potential abuse.

@drewstone drewstone changed the title Time travel up 2 New Treasury reward module updates Jun 5, 2020
@drewstone drewstone requested a review from dillchen June 10, 2020 21:06
@dillchen dillchen requested a review from raykyri June 17, 2020 06:07
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.

1 participant