polkadot-fellows / RFCs

Proposals for change to standards administered by the Fellowship.
https://polkadot-fellows.github.io/RFCs/
Creative Commons Zero v1.0 Universal
109 stars 47 forks source link

Increase the maximum length of identity raw data values from 32 bytes #76

Closed ltfschoen closed 3 months ago

bkchr commented 4 months ago

The identity is now generic and configured in the runtime. Generally not sure we need a RFC for such a change. I think it is fine to just do this as a pr and convincing of the maintainers.

CC @joepetrowski

ggwpez commented 4 months ago

I think Joe is OOO, but currently we re-use this Data type in the new Rococo people config and partly in Polkadot.
It should be enough to make that 32 there generic as Get<u32>.

Then in a second change you can increase it for the Polkadot runtime config.

joepetrowski commented 4 months ago

Yeah I agree with @ggwpez and @bkchr , should be a PR to polkadot-sdk to make that value configurable and then we can propose the change in the pallet config.

bkchr commented 3 months ago

@ltfschoen are you okay with the proposed steps?

ltfschoen commented 3 months ago

@ltfschoen are you okay with the proposed steps?

yes i agree with @ggwpez, @joepetrowski, and yourself

bkchr commented 3 months ago

So this can be closed?

ltfschoen commented 3 months ago

Closed and replaced with Polkadot SDK issue https://github.com/paritytech/polkadot-sdk/issues/3977