Skip to content

Update pallet nomination pool to support Block Number Provider#6715

Merged
muharem merged 8 commits intoparitytech:masterfrom
dharjeezy:dami/pallet_nomination_pool_block_number_provider
Feb 8, 2025
Merged

Update pallet nomination pool to support Block Number Provider#6715
muharem merged 8 commits intoparitytech:masterfrom
dharjeezy:dami/pallet_nomination_pool_block_number_provider

Conversation

@dharjeezy
Copy link
Copy Markdown
Contributor

@dharjeezy dharjeezy commented Dec 1, 2024 •
edited
Loading

This PR introduces BlockNumberProvider config for the nomination pool pallet.
closes part of #6297

Polkadot address: 12GyGD3QhT4i2JJpNzvMf96sxxBLWymz4RdGCxRH5Rj5agKW

@dharjeezy dharjeezy requested a review from a team as a code owner December 1, 2024 02:12
gui1117
gui1117 approved these changes Dec 1, 2024
View reviewed changes
Copy link
Copy Markdown
Contributor

@gui1117 gui1117 left a comment

Choose a reason for hiding this comment

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

Looks good to me,

gui1117
gui1117 reviewed Dec 2, 2024
View reviewed changes
@gui1117 gui1117 added the T2-pallets This PR/Issue is related to a particular pallet. label Dec 2, 2024
@dharjeezy dharjeezy mentioned this pull request Jan 14, 2025
Closed
muharem
muharem approved these changes Jan 24, 2025
View reviewed changes
prdoc/pr_6715.prdoc Outdated
- audience: Runtime Dev
description: |
This PR makes the block number provider used in the nomination pool pallet configurable. Before this PR, nomination pool pallet always used the system block number,
with this PR some runtime can opt to use the relay chain block number instead.
Copy link
Copy Markdown
Contributor

@muharem muharem Jan 24, 2025

Choose a reason for hiding this comment

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

same comment as in the other PR.

@dharjeezy
Merge remote-tracking branch 'origin/master' into dami/pallet_nominat…
3e9f335 
…ion_pool_block_number_provider

# Conflicts:
#	substrate/frame/nomination-pools/test-transfer-stake/src/mock.rs
@dharjeezy
nit
b432eef
@muharem
Copy link
Copy Markdown
Contributor

muharem commented Feb 7, 2025

@dharjeezy please remove this empty file - substrate/frame/nomination-pools/test-transfer-stake/src/mock.rs

@muharem muharem added this pull request to the merge queue Feb 8, 2025
Merged via the queue into paritytech:master with commit 10b77c9 Feb 8, 2025
140 of 143 checks passed
@gui1117 gui1117 mentioned this pull request Feb 11, 2025
Merged
@ggwpez
Copy link
Copy Markdown
Member

ggwpez commented Feb 11, 2025

/tip small

@substrate-tip-bot
Copy link
Copy Markdown

substrate-tip-bot bot commented Feb 11, 2025

@dharjeezy Contributor did not properly post their account address.

Make sure the pull request description (or user bio) has: "{network} address: {address}".

@dharjeezy
Copy link
Copy Markdown
Contributor Author

dharjeezy commented Feb 11, 2025

@dharjeezy Contributor did not properly post their account address.

Make sure the pull request description (or user bio) has: "{network} address: {address}".

i have updated my address @ggwpez

@ggwpez
Copy link
Copy Markdown
Member

ggwpez commented Feb 11, 2025

/tip small

@substrate-tip-bot
Copy link
Copy Markdown

substrate-tip-bot bot commented Feb 11, 2025

@ggwpez A referendum for a small (20 DOT) tip was successfully submitted for @dharjeezy (12GyGD3QhT4i2JJpNzvMf96sxxBLWymz4RdGCxRH5Rj5agKW on polkadot).

Referendum number: 1431.
tip

@substrate-tip-bot
Copy link
Copy Markdown

substrate-tip-bot bot commented Feb 11, 2025

The referendum has appeared on Polkassembly.

clangenb pushed a commit to clangenb/polkadot-sdk that referenced this pull request Feb 19, 2025
@dharjeezy @clangenb
Update pallet nomination pool to support Block Number Provider (parit…
69610e9 
…ytech#6715)

This PR introduces BlockNumberProvider config for the nomination pool
pallet.
closes part of paritytech#6297
athei added a commit to athei/runtimes that referenced this pull request May 5, 2025
@athei
Add BlockNumberProvider
4b3e9db 
paritytech/polkadot-sdk#6338
paritytech/polkadot-sdk#6623
paritytech/polkadot-sdk#6621
paritytech/polkadot-sdk#6715
paritytech/polkadot-sdk#6446
paritytech/polkadot-sdk#7441
paritytech/polkadot-sdk#5723
@ggwpez ggwpez mentioned this pull request Jun 4, 2025
Merged
7 tasks
@github-actions github-actions bot mentioned this pull request Jun 10, 2025
Closed
@github-actions github-actions bot mentioned this pull request Jul 16, 2025
Closed
@github-actions github-actions bot mentioned this pull request Jul 16, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gui1117 gui1117 gui1117 approved these changes

@muharem muharem muharem approved these changes

Assignees

No one assigned

Labels

T2-pallets This PR/Issue is related to a particular pallet.

Projects

[preview] release tracker
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@dharjeezy @muharem @ggwpez @gui1117 @pgherveou