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.

Sign up for GitHub

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

Jump to bottom

Ensure Supra-specific configuration settings are stored on-chain #83

Closed
isaacdoidge opened this issue Sep 26, 2024 · 2 comments · Fixed by #103
Closed

Ensure Supra-specific configuration settings are stored on-chain #83

isaacdoidge opened this issue Sep 26, 2024 · 2 comments · Fixed by #103
Assignees
@isaacdoidge @axiongsupra
Labels
enhancement New feature or request

Comments

@isaacdoidge
Copy link

We need to store our protocol configuration settings on-chain so that we can update them via governance.
@isaacdoidge isaacdoidge added the enhancement New feature or request label Sep 26, 2024
@isaacdoidge isaacdoidge self-assigned this Sep 26, 2024
isaacdoidge added a commit that referenced this issue Sep 26, 2024
@isaacdoidge
[#83] Added supra_config.move to the Supra Move Framework and genesis…
05c67bf 
… flow. Also added a script to run the unit tests that we care about.
@isaacdoidge isaacdoidge mentioned this issue Sep 26, 2024
Merged
isaacdoidge added a commit that referenced this issue Sep 26, 2024
@isaacdoidge
Ensured that the Supra protocol configuration settings are stored in …
b710f2d 
…the Move state. (#84)

* [#83] Added supra_config.move to the Supra Move Framework and genesis flow. Also added a script to run the unit tests that we care about.
@isaacdoidge
Copy link
Author

@axiongsupra, can you please double check that the new spec that I added in #84?

@axiongsupra axiongsupra self-assigned this Oct 7, 2024
@axiongsupra axiongsupra mentioned this issue Oct 8, 2024
Merged
@axiongsupra axiongsupra linked a pull request Oct 8, 2024 that will close this issue
Confirm spec works as expected and update the documentation #103
Merged
@axiongsupra
Copy link

I don't understand why my link did not close this issue automatically...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@isaacdoidge isaacdoidge

@axiongsupra axiongsupra

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Confirm spec works as expected and update the documentation Entropy-Foundation/aptos-core
2 participants
@isaacdoidge @axiongsupra