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

Add field names to pallet Event variants #9903

Open
ascjones opened this issue Sep 30, 2021 · 1 comment
Open

Add field names to pallet Event variants #9903

ascjones opened this issue Sep 30, 2021 · 1 comment
Labels
Z1-easy Can be fixed primarily by duplicating and adapting code by an intermediate coder

Comments

@ascjones
Copy link
Contributor

As per #9896, we can convert other pallets to use struct variants with named fields.

e.g. Instantiated(T::AccountId, T::AccountId) becomes Instantiated { deployer: T::AccountId, contract: T::AccountId }.

These field names and their respective comments are in turn reflected in the metadata for a better client experience.
@ascjones ascjones added the Z1-easy Can be fixed primarily by duplicating and adapting code by an intermediate coder label Sep 30, 2021
@Wizdave97
Copy link
Contributor

PIcking this up

@Wizdave97 Wizdave97 mentioned this issue Oct 11, 2021
Merged
@ascjones ascjones mentioned this issue Oct 22, 2021
Merged
7 tasks
@ayevbeosa ayevbeosa mentioned this issue Nov 4, 2021
Merged
@Doordashcon Doordashcon mentioned this issue Nov 6, 2021
Merged
@Doordashcon Doordashcon mentioned this issue Nov 13, 2021
Merged
@ECJ222 ECJ222 mentioned this issue Nov 17, 2021
Merged
@stanly-johnson stanly-johnson mentioned this issue Dec 25, 2021
Closed
@hieronx hieronx mentioned this issue Jul 18, 2022
Closed
6 tasks
@seadanda seadanda mentioned this issue Sep 25, 2023
Merged
@davidk-pt davidk-pt mentioned this issue Aug 8, 2024
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Z1-easy Can be fixed primarily by duplicating and adapting code by an intermediate coder
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ascjones @Wizdave97