Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Update Currency Const Use#6386

Closed
joepetrowski wants to merge 1 commit intogav-tweak-gov2from
joe-update-currency-use
Closed

Update Currency Const Use#6386
joepetrowski wants to merge 1 commit intogav-tweak-gov2from
joe-update-currency-use

Conversation

@joepetrowski
Copy link
Copy Markdown
Contributor

@joepetrowski joepetrowski commented Dec 5, 2022

RE #6372 (comment)

This change is quite intrusive and I'm worried of missing a use somewhere. I think it's safer to just keep pub const QUID: Balance = UNITS / 300; and update the deposits in the governance tracks.

TODO if we keep this PR:

  • Cumulus companion (System parachains use these constants too)

@github-actions github-actions bot added the A3-in_progress Pull request is in progress. No review needed at this stage. label Dec 5, 2022
@joepetrowski joepetrowski mentioned this pull request Dec 5, 2022
2 tasks
@gavofyork gavofyork closed this Dec 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A3-in_progress Pull request is in progress. No review needed at this stage.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants