-
Notifications
You must be signed in to change notification settings - Fork 2.6k
CountedNMap implementation #10621
CountedNMap implementation #10621
Conversation
9205bdf
to
eac9616
Compare
@bkchr , @shawntabrizi don't want to be tactless, but can we proceed with this PR, it will be helpful to solve this issue paritytech/polkadot-sdk#316. Also after if this PR will be approved and merged I will prepare the same update for the |
This PR seems valuable, yet hat slipped through the crack. Partially due to lack of labeling and project assignment from our side. Hopefully will get some review soon. Thanks @Mr-Leshiy! |
kindly ping |
bot fmt |
@juangirini https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3298354 was started for your command Comment |
@juangirini Command |
@Mr-Leshiy please note that there are some formatting issues that |
@juangirini |
Could it be merged? this is really an awesome feature |
bot merge |
Error: Statuses failed for 4c29a0e |
bot rebase |
Rebased |
bot merge |
Waiting for commit status. |
/tip medium |
@Mr-Leshiy Contributor did not properly post their account address. Make sure the pull request description (or user bio) has: "{network} address: {address}". |
@Mr-Leshiy interested in a tip? |
@kianenigma would be great :) |
/tip medium |
@juangirini A medium (80 DOT) tip was successfully submitted for @Mr-Leshiy (16GBceb9xiWB1R5YQGRfEYDZbWcatmLz8MPd6KZYpy4qfV7B on polkadot). https://polkadot.js.org/apps/?rpc=wss%3A%2F%2Frpc.polkadot.io#/referenda |
add
CountedStorageNMap
implementation which is similar asCountedStorageMap
forStorageMap
but for theStorageNMap
#10602
polkadot address: 16GBceb9xiWB1R5YQGRfEYDZbWcatmLz8MPd6KZYpy4qfV7B