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

pallet-core-fellowship: import an unimported member on approve #2883

Merged
merged 3 commits into from
Jan 9, 2024

Conversation

muharem
Copy link
Contributor

@muharem muharem commented Jan 9, 2024

To align with the documentation of the approve call, we import an unimported member on approval.

No changes have been made to the benchmarks as they already cover the worst-case scenario.

@muharem muharem added the T1-FRAME This PR/Issue is related to core FRAME, the framework. label Jan 9, 2024
@muharem muharem requested review from a team January 9, 2024 04:30
@muharem muharem changed the title core fellowship pallet: import not tracked member on approve pallet-core-fellowship: import not tracked member on approve Jan 9, 2024
@muharem muharem changed the title pallet-core-fellowship: import not tracked member on approve pallet-core-fellowship: import an unimported member on approve Jan 9, 2024
@ggwpez ggwpez merged commit d288c81 into master Jan 9, 2024
125 checks passed
@ggwpez ggwpez deleted the muharem-core-fellowship-import branch January 9, 2024 11:36
github-merge-queue bot pushed a commit that referenced this pull request Jan 17, 2024
EgorPopelyaev pushed a commit that referenced this pull request Jan 17, 2024
bgallois pushed a commit to duniter/duniter-polkadot-sdk that referenced this pull request Mar 25, 2024
…ytech#2883)

To align with the documentation of the approve call, we import an
unimported member on approval.

No changes have been made to the benchmarks as they already cover the
worst-case scenario.
bgallois pushed a commit to duniter/duniter-polkadot-sdk that referenced this pull request Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T1-FRAME This PR/Issue is related to core FRAME, the framework.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants