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

Make outer call/origin non breaking change when one pallet removes one call/origin #6855

@gui1117

Description

@gui1117

currently when one pallet removes it calls or its origin, or if a runtime removes a pallet, then all indices are changed.
This is annoying because calls are usually expected not to change and same for origin (which is stored on chain by scheduler).

(also maybe event should benefit from it as well.)

Instead in the codec implementation of event/calls/origin we should use (when decoding the variant index) a fixed index given to construct_runtime.

Ideally I was thinking first using ModuleToIndex and allow user to set the index of a module. But this is not perfect because currently some pallet have their call at index n and their origin at index p. Thus in order not to break those types anymore, construct_runtime should allow to configure them independently.

I have no idea of syntax yet.

Metadata

Metadata

Assignees

No one assigned

    Labels

    J0-enhancementAn additional feature request.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions