-
Notifications
You must be signed in to change notification settings - Fork 648
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
Implement try_append for StorageNMap #5745
Conversation
LGTM |
assert_ok!(FooQuadMap::try_append((2, 1, 1, 1), 5)); | ||
assert_eq!( | ||
FooQuadMap::get((2, 1, 1, 1)).unwrap(), | ||
BoundedVec::<u32, ConstU32<7>>::try_from(vec![4, 5]).unwrap(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code above uses the same syntax, so i think this is fine
/tip medium @CJ13th follow the instructions below to receive a tip |
@CJ13th Contributor did not properly post their account address. Make sure the pull request description (or user bio) has: "{network} address: {address}". |
e0766bb
Thanks all for reviewing and merging. @shawntabrizi I have updated the pull request description and my bio to include my DOT address. |
/tip medium |
The referendum has appeared on Polkassembly. |
Description
Closes #5722
Added an implementation of the
try_append
functionality which is present on the other storage map types but currently missing from StorageNMap.polkadot address: 12VNFoH4iy4227mpTPvUbUbAnYv7of6LhWcD6EUik3VfFHbD