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

[staking] skip voters that have voter weight = 0 #13939

Closed
wants to merge 2 commits into from
Closed

[staking] skip voters that have voter weight = 0 #13939

wants to merge 2 commits into from

Conversation

gpestana
Copy link
Contributor

Closes #13938

@gpestana gpestana added A3-in_progress Pull request is in progress. No review needed at this stage. B0-silent Changes should not be mentioned in any release notes C3-medium PR touches the given topic and has a medium impact on builders. D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed. T1-runtime This PR/Issue is related to the topic “runtime”. labels Apr 18, 2023
@gpestana gpestana self-assigned this Apr 18, 2023
@gpestana gpestana requested a review from a team April 18, 2023 09:36
@gpestana gpestana marked this pull request as draft April 18, 2023 09:36
@gpestana gpestana changed the title Minimum active stake should not be zero and decoding error handling [staking] skip voters that have voter weight = 0 and ensure min active stake is higher than min nominator bond Apr 18, 2023
@gpestana gpestana changed the title [staking] skip voters that have voter weight = 0 and ensure min active stake is higher than min nominator bond [staking] skip voters that have voter weight = 0 Apr 18, 2023
@gpestana gpestana closed this Apr 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@gpestana gpestana

Labels
A3-in_progress Pull request is in progress. No review needed at this stage. B0-silent Changes should not be mentioned in any release notes C3-medium PR touches the given topic and has a medium impact on builders. D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed. T1-runtime This PR/Issue is related to the topic “runtime”.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

MinimumActivesStake reported as 0
1 participant
@gpestana