-
Notifications
You must be signed in to change notification settings - Fork 689
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
Vision: Support Migrating Encoded Calls #338
Comments
relates to #353 |
Hmm what support do you want to have for this? You need the old |
Jep, we'd want some (at least moderately) easy way to get the deprecated Call enum. |
We entertained the idea of using a call hash instead of a call enum, but this would result in a larger payload size for all transactions. A solution would just be to allow us to control the call index of the extrinsics in a pallet. We are already able to control the module index too, so together, things would remain deterministic and non-breaking. |
You would still (on top of that) want to be able to determine removed indices as well as changed ones to migrate properly. |
* First cut of erc721 pallet * NFT pallet (wip) * Tests not working yet * Nft pallet tests * Coupling erc721 and nft pallets (wip) * Take token info to primitives * Token data, mocking ERC721, some tests * More tests; Fixed compilation of serializables in runtime code * Integrate pallets in runtime and related changes * Added ERC721App with lock feature * Update deprecated code * Compilation fixes * Update tests and fix encoding * Added unlock feature * added validation and documentation * fix: compilation * tests: add test cases to nft pallet * tests: add transfer assertions * refactor: more descriptive token and token id names * feat: add access control to erc721 contract * fix: solidity compilation, move migration of ERC721 into seperate file * tests: fix deployment of ERC1820 registry * fix: unlocking, fix tests * tests: add unhappy path test cases for erc721 tokens * fix: compilation * fix: compilation * tests: add ERC721 e2e test * refactor: simplify TestToken721 * fix: remove not needed NFT ownership assertion * fix: test/README.md * chore: rename erc721 migration * fix: erc721 app address * fix: TokenId type * refactor: remove unnecessary assertion in ERC721 unlock solidity contract * chore: remove question in comment * tests: add example to fetch Substrate tokenId from Ethereum tokenContract and tokenId to ERC721 e2e test Co-authored-by: Bhushan Lodha <bhushanlodha@gmail.com> Co-authored-by: Vincent Geddes <vincent.geddes@hey.com> Co-authored-by: Philip Stanislaus <6912756+philipstanislaus@users.noreply.github.com>
When changing the encoding of calls (e.g. changing pallet indices or extrinsic ordering) we need to migrate encoded calls (e.g. in Scheduler or Democracy) to the new format.
Currently we can only hope that they are not affected by runtime upgrades or clear them in case of incompatibility.
The text was updated successfully, but these errors were encountered: