Skip to content

Conversation

@furszy
Copy link

@furszy furszy commented May 18, 2021

Cleaning pre-v5 consensus guards for the proposal start block and max payment amount validation.

@furszy furszy self-assigned this May 18, 2021
@furszy furszy added this to the 6.0.0 milestone May 18, 2021
@furszy furszy added the Cleanup label May 18, 2021
@random-zebra
Copy link

We cannot do this yet, or we risk forking the network, as we have still active proposals that were submitted before V5 (thus with old rules). Note that the new rules apply only if the proposal's block-start is after v5 enforcement.

There are 3 of these proposals. Two of them will expire after then next cycle. But the last one still needs 4 more cycles (it has a payment count of 12, which is not even allowed by the new rules).

@furszy
Copy link
Author

furszy commented May 18, 2021

No need to merge this one now, let's leave it open until we can do it.

@furszy furszy modified the milestones: 5.3.0, 6.0.0 Aug 4, 2021
@furszy furszy force-pushed the 2021_clean_budget_v5_rules branch from 9c0f399 to 3a9476c Compare December 17, 2021 13:35
Copy link

@random-zebra random-zebra left a comment

Choose a reason for hiding this comment

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

utACK 3a9476c

Copy link
Collaborator

@Fuzzbawls Fuzzbawls left a comment

Choose a reason for hiding this comment

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

ACK 3a9476c

@furszy furszy merged commit d277bdf into PIVX-Project:master Dec 18, 2021
@Fuzzbawls Fuzzbawls modified the milestones: 6.0.0, 5.5.0 Sep 11, 2022
@furszy furszy deleted the 2021_clean_budget_v5_rules branch November 29, 2022 14:36
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.

3 participants