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

Reduce Bounty Payout Delay #386

Merged
merged 3 commits into from
Jul 17, 2024
Merged

Conversation

joepetrowski
Copy link
Contributor

The bounty payout delay originally existed to provide time for the Council to remove curators of a bounty if they were not acting in accordance with its existence. The Council motion time was 7 days, and thus the bounty delay was 8 days to give the Council time to remove curators and void payouts prior to a bounty being claimable.

With OpenGov tracks being 28 days, I don't see this as anything more than a PITA for curators/awardees. Does anyone have motivation for this delay to be in effect in the current setup? (cc @muharem , @shawntabrizi )

@shawntabrizi
Copy link
Member

Your logic to remove it makes sense to me.

Ideally we could still prevent bounties from being raided by curators, but I guess this will happen through denying "top ups" rather than through the spend.

@joepetrowski
Copy link
Contributor Author

/merge

@fellowship-merge-bot
Copy link
Contributor

Enabled auto-merge in Pull Request

Available commands
  • /merge: Enables auto-merge for Pull Request
  • /merge cancel: Cancels auto-merge for Pull Request
  • /merge help: Shows this menu

For more information see the documentation

@fellowship-merge-bot fellowship-merge-bot bot enabled auto-merge (squash) July 17, 2024 05:47
@fellowship-merge-bot fellowship-merge-bot bot merged commit 2c13e08 into main Jul 17, 2024
43 checks passed
@joepetrowski joepetrowski deleted the joe-reduce-bounty-delay branch July 17, 2024 09:01
@muddlebee
Copy link

@joepetrowski which Runtimes version this will be part of? I dont see it in changelog of upcoming ones 👀

@joepetrowski
Copy link
Contributor Author

image

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.

5 participants