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

Validators missing from api.derive.staking.electedInfo #6004

Closed
3 of 10 tasks
youPickItUp opened this issue Oct 22, 2024 · 4 comments · Fixed by #6011
Closed
3 of 10 tasks

Validators missing from api.derive.staking.electedInfo #6004

youPickItUp opened this issue Oct 22, 2024 · 4 comments · Fixed by #6011

Comments

@youPickItUp
Copy link

  • I'm submitting a ...
  • Bug report
  • Feature request
  • Support request
  • Other
  • What is the current behavior and expected behavior?

Currently, validators without other nominators aren't returned from api.derive.staking.electedInfo as nextElected.
Expectedly, validators without other nominators should be returned from api.derive.staking.electedInfo as nextElected.

switchMap(({ currentEra }) => api.query.staking.erasStakersPaged.keys(currentEra)),

erasStakersPaged contains only validators being nominated by others.

  • What is the motivation for changing the behavior?

I think this is the correct behavior, I'd be glad to stand corrected.

  • Please tell us about your environment:
  • Version: 13.2.1, but master seems to have the same issue

  • Environment:

    • Node.js
    • Browser
    • Other (limited support for other environments)
  • Language:

    • JavaScript
    • TypeScript (include tsc --version)
    • Other
@TarikGul
Copy link
Member

erasStakersPaged contains only validators being nominated by others.

It's been a while since I looked at the rust code for this, but this very well could be correct. I am just not sure what call this would be replaced by but I will have a look!

@youPickItUp
Copy link
Author

I think it should be erasStakersOverview

@TarikGul
Copy link
Member

I think it should be erasStakersOverview

Yea I agree, I'll get a PR up shortly.

@polkadot-js-bot
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue if you think you have a related problem or query.

@polkadot-js polkadot-js locked as resolved and limited conversation to collaborators Nov 5, 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 a pull request may close this issue.

3 participants