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

Update inflation spec #613

Merged
merged 13 commits into from
Oct 20, 2022
Merged

Update inflation spec #613

merged 13 commits into from
Oct 20, 2022

Conversation

brentstone
Copy link
Collaborator

@brentstone brentstone commented Oct 14, 2022

This also seems like a better place than Slack to discuss some of the changes.

The current procedure of #388 stores the token ratios rather than the errors (which are a function of the ratios), so these changes reflect that now here. Can easily decide to store the errors instead if desired.

@brentstone brentstone marked this pull request as draft October 14, 2022 18:05
@juped juped added documentation Improvements or additions to documentation spec labels Oct 14, 2022
Copy link
Contributor

@cwgoes cwgoes left a comment

Choose a reason for hiding this comment

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

Thanks for the updates - one minor quibble but otherwise this is clearer - thanks!

documentation/specs/src/economics/inflation-system.md Outdated Show resolved Hide resolved
@brentstone brentstone marked this pull request as ready for review October 17, 2022 13:51
@juped juped merged commit 72feea5 into main Oct 20, 2022
@juped juped deleted the brent/update-inflation-spec branch October 20, 2022 12:09
@brentstone brentstone mentioned this pull request Oct 25, 2022
15 tasks
phy-chain pushed a commit to phy-chain/namada that referenced this pull request Mar 1, 2024
* feat: new UI for passphrase

* fix: proper seed for zip32 flow

* fix: fixed lint issue for ToggleButton

* chore: cleanup unused commented out component

* test: e2e tests for passphrase support

* fix: make passphrase in storage optional
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation spec
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants