Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

[pallet_collective] Usage of Vec instead of BoundedVec #12574

Closed
Chralt98 opened this issue Oct 28, 2022 · 2 comments
Closed

[pallet_collective] Usage of Vec instead of BoundedVec #12574

Chralt98 opened this issue Oct 28, 2022 · 2 comments
Labels
J2-unconfirmed Issue might be valid, but it’s not yet known.

Comments

@Chralt98
Copy link

I found an instance, where other pallets are using a BoundedVec, but pallet_collective does not.

substrate/frame/collective/src/lib.rs

Line 221 in 6195ef4

pub members: Vec<T::AccountId>,

@github-actions github-actions bot added the J2-unconfirmed Issue might be valid, but it’s not yet known. label Oct 28, 2022
@dharjeezy
Copy link
Contributor

I will like to take this issue up and fix. @Chralt98

@ggwpez
Copy link
Member

ggwpez commented Oct 28, 2022

The pallet bounding is ongoing here paritytech/polkadot-sdk#323
In this case it is in the genesis config and therefore not a problem anyway.

@ggwpez ggwpez closed this as completed Oct 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
J2-unconfirmed Issue might be valid, but it’s not yet known.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ggwpez @dharjeezy @Chralt98