Skip to content
This repository has been archived by the owner on May 21, 2024. It is now read-only.

feat: update pallet-uniques string limits. #297 #298

Merged
merged 1 commit into from
Oct 6, 2023
Merged

feat: update pallet-uniques string limits. #297 #298

merged 1 commit into from
Oct 6, 2023

Conversation

peterwht
Copy link
Contributor

@peterwht peterwht commented Sep 29, 2023 •
edited
Loading

See the following for more details: #297.

Update the pallet-uniques string limits to match Polkadot's Asset hub.

https://github.com/paritytech/polkadot-sdk/blob/master/cumulus/parachains/runtimes/assets/asset-hub-polkadot/src/lib.rs#L683

	pub const StringLimit: u32 = 128;
	pub const KeyLimit: u32 = 32;
	pub const ValueLimit: u32 = 64;

@peterwht peterwht self-assigned this Oct 4, 2023
@stiiifff stiiifff merged commit c067c9b into main Oct 6, 2023
3 checks passed
@stiiifff stiiifff deleted the pw/update-uniques-string-limit branch October 6, 2023 09:14
@stiiifff stiiifff mentioned this pull request Oct 6, 2023
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@stiiifff stiiifff stiiifff approved these changes

@hbulgarini hbulgarini Awaiting requested review from hbulgarini

Assignees

@peterwht peterwht

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@peterwht @stiiifff