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.

Sign up for GitHub

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

Jump to bottom

Add size bounds to pallet election-provider-multi-phase #6348

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Add size bounds to pallet election-provider-multi-phase #6348

wants to merge 2 commits into from

Conversation

re-gius
Copy link
Contributor

@re-gius re-gius commented Nov 4, 2024 •
edited
Loading

Removing without_storage_info and adding bounds on the stored types for pallet election-provider-multi-phase - issue #6289

@re-gius re-gius mentioned this pull request Nov 4, 2024
Open
5 tasks
@re-gius re-gius self-assigned this Nov 4, 2024
@re-gius re-gius added T2-pallets This PR/Issue is related to a particular pallet. T12-benchmarks This PR/Issue is related to benchmarking and weights. labels Nov 4, 2024
@kianenigma
Copy link
Contributor

Some types in this pallet may only be bound-able after we review #6034, I think. Please check with @gpestana

@re-gius
Copy link
Contributor Author

re-gius commented Nov 5, 2024

Some types in this pallet may only be bound-able after we review #6034, I think. Please check with @gpestana

PR is currently suspended waiting for #6034 to be merged.

@gpestana
Copy link
Contributor

@re-gius thanks for working on this. Given #6034, I think it would be easier to open this PR against the #6034 branch or to wait for it to be merged. There are some bounds introduced in that PR that may overlap with this PR, but overall I think it should simplify your PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@re-gius re-gius

Labels
T2-pallets This PR/Issue is related to a particular pallet. T12-benchmarks This PR/Issue is related to benchmarking and weights.
Projects
AHM
Status: In progress
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@re-gius @kianenigma @gpestana