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

Showing waiting, active and inactive nominations on the Accounts page #10998

Merged
merged 6 commits into from
Nov 4, 2024

Conversation

piggydoughnut
Copy link
Contributor

@piggydoughnut piggydoughnut commented Oct 11, 2024

Staking Pallet changes for v14 applied.
Backwards compatibility ensured.

deprecated calls:
erasStakers - deprecated in v14
stakers - deprecated earlier

new calls:
erasStakersPaged - used in a query.derive.staking.eraExposure

Note:
query.derive.staking.eraExposure is a really expensive call, it delays showing the numbers of validators by a few seconds. I added a spinner there.

Resolves #10804

@piggydoughnut piggydoughnut marked this pull request as draft October 11, 2024 10:45
@piggydoughnut piggydoughnut marked this pull request as ready for review October 18, 2024 10:36
stashId: string;
slashes: Option<SlashingSpans>[];
nominees: string[];
activeEra: DeriveSessionIndexes['activeEra'] | undefined;
Copy link
Member

Choose a reason for hiding this comment

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

Probably better to set this to EraIndex instead of DeriveSessionIndexes['activeEra'] as it will be more explicit.

Copy link
Member

@TarikGul TarikGul left a comment

Choose a reason for hiding this comment

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

Few nits, overall amazing!

Copy link
Member

@TarikGul TarikGul left a comment

Choose a reason for hiding this comment

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

Great job!

@TarikGul TarikGul merged commit 3ec308f into master Nov 4, 2024
5 checks passed
@TarikGul TarikGul deleted the dm-10804-bug-waiting-nominations branch November 4, 2024 13:45
@polkadot-js-bot
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@polkadot-js polkadot-js locked as resolved and limited conversation to collaborators Nov 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

All nominations show up as "waiting"
3 participants