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

[Pools] New runtime api that returns the pot accounts associated with the pool #6357

Merged
merged 6 commits into from
Nov 6, 2024

Conversation

Ank4n
Copy link
Contributor

@Ank4n Ank4n commented Nov 5, 2024

closes #6358

Adds the following runtime api to pallet-nomination-pools.
pool_accounts(pool_id): Returns (bonded_account, reward_account) associated with the pool_id.

cc: @rossbulat

@Ank4n Ank4n requested a review from koute as a code owner November 5, 2024 00:52
@Ank4n Ank4n added T2-pallets This PR/Issue is related to a particular pallet. T4-runtime_API This PR/Issue is related to runtime APIs. labels Nov 5, 2024
@Ank4n
Copy link
Contributor Author

Ank4n commented Nov 5, 2024

bot fmt

@command-bot
Copy link

command-bot bot commented Nov 5, 2024

@Ank4n https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7693500 was started for your command "$PIPELINE_SCRIPTS_DIR/commands/fmt/fmt.sh". Check out https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment bot cancel 16-01a67032-ac4b-4d3b-879b-afe5682afae5 to cancel this command or bot cancel to cancel all commands in this pull request.

@command-bot
Copy link

command-bot bot commented Nov 5, 2024

@Ank4n Command "$PIPELINE_SCRIPTS_DIR/commands/fmt/fmt.sh" has finished. Result: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7693500 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/7693500/artifacts/download.

@Ank4n Ank4n requested review from gpestana and a team November 5, 2024 01:36
Copy link

@rossbulat rossbulat left a comment

Choose a reason for hiding this comment

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

Excellent, this will help applications a lot 🫡

Copy link
Contributor

@kianenigma kianenigma left a comment

Choose a reason for hiding this comment

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

This is good to add, but we will probably migrate it to #4722 🙈

we can keep the existing APIs for backwards compatibility for some time.

@Ank4n Ank4n added this pull request to the merge queue Nov 6, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 6, 2024
@Ank4n Ank4n enabled auto-merge November 6, 2024 19:58
@Ank4n Ank4n added this pull request to the merge queue Nov 6, 2024
Merged via the queue into master with commit 1f381f6 Nov 6, 2024
194 of 197 checks passed
@Ank4n Ank4n deleted the ankan/np-runtime-api-pool-account branch November 6, 2024 23:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T2-pallets This PR/Issue is related to a particular pallet. T4-runtime_API This PR/Issue is related to runtime APIs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Nomination pools Runtime API: Get a pool stash and reward address
5 participants