-
Notifications
You must be signed in to change notification settings - Fork 19
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
Identity staking follow up #2976
Identity staking follow up #2976
Conversation
the parentchain storage and idgraphs
…-periodic-update-staking-amount-of-idgraph
…g-amount-of-idgraph
…g-amount-of-idgraph
|
||
#[pallet::call_index(10)] | ||
#[pallet::weight((195_000_000, DispatchClass::Normal))] | ||
pub fn complete_reward_distribution(origin: OriginFor<T>) -> DispatchResultWithPostInfo { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm strange - where is the token distribution process? (I thought it would be done within this extrinsic)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is just to trigger an event that marks the distribution for the round as completed. (Currently we don't do anything with the token_staking_amount
, will there be another phase/issue to come?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No I mean "distribution for the round as completed" should be triggered by the worker - when it finishes with staking amount upload for all accounts, it should trigger the distribution
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have updated the code to make the distribution on update_token_staking_amount
, this extrinsic will be kept to signal that the distribution for the round is completed.
…-periodic-update-staking-amount-of-idgraph
…en_staking_amount
…g-amount-of-idgraph
This reverts commit 2f55394.
This PR adds a way to do periodic updates for the staking amount of IDGraphs.
More context in the issue.