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

Support storage_alias for CountedStorageMap #13353

Closed
ggwpez opened this issue Feb 9, 2023 · 3 comments · Fixed by #13366
Closed

Support storage_alias for CountedStorageMap #13353

ggwpez opened this issue Feb 9, 2023 · 3 comments · Fixed by #13366
Assignees
@ruseinov
Labels
J0-enhancement An additional feature request. U3-nice_to_have Issue is worth doing eventually. Z2-medium Can be fixed by a coder with good Rust knowledge but little knowledge of the codebase.

Comments

@ggwpez
Copy link
Member

ggwpez commented Feb 9, 2023 •
edited
Loading

storage_alias should support CountedStorageMap but does not.
@ggwpez ggwpez added J0-enhancement An additional feature request. U3-nice_to_have Issue is worth doing eventually. Z2-medium Can be fixed by a coder with good Rust knowledge but little knowledge of the codebase. labels Feb 9, 2023
@bkchr
Copy link
Member

bkchr commented Feb 9, 2023

Which ticket? Why should we remove this?

@ruseinov
Copy link
Contributor

Which ticket? Why should we remove this?

Disregard, it stuck in my mind, but it had to do with storage getters.

@ruseinov
Copy link
Contributor

Will take this on, might be needing this soon.

@ruseinov ruseinov self-assigned this Feb 12, 2023
@ruseinov ruseinov mentioned this issue Feb 12, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ruseinov ruseinov

Labels
J0-enhancement An additional feature request. U3-nice_to_have Issue is worth doing eventually. Z2-medium Can be fixed by a coder with good Rust knowledge but little knowledge of the codebase.
Projects
Runtime / FRAME
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Feature] Introduce storage_alias for CountedStorageMap paritytech/substrate
3 participants
@ruseinov @bkchr @ggwpez