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

Use raw amount for staking issuance parameter #4687

Merged
merged 3 commits into from
Jul 1, 2024

Conversation

cronokirby
Copy link
Contributor

We were scaling this before using the default metadata, which is more brittle, and less precise.

Checklist before requesting a review

  • If this code contains consensus-breaking changes, I have added the "consensus-breaking" label. Otherwise, I declare my belief that there are not consensus-breaking changes, for the following reason:

    This will change how the parameter is interpreted.

@cronokirby cronokirby added the consensus-breaking breaking change to execution of on-chain data label Jul 1, 2024
We were scaling this before using the default metadata, which is more brittle, and less precise.
@cronokirby cronokirby force-pushed the using-raw-amount-for-staking-issuance-param branch from dae653f to 16ff690 Compare July 1, 2024 21:44
@erwanor erwanor self-requested a review July 1, 2024 22:02
@erwanor erwanor merged commit 800d93d into main Jul 1, 2024
13 checks passed
@erwanor erwanor deleted the using-raw-amount-for-staking-issuance-param branch July 1, 2024 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
consensus-breaking breaking change to execution of on-chain data
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants