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 #10184

Conversation

ayevbeosa
Copy link
Contributor

@ayevbeosa ayevbeosa commented Nov 4, 2021

This pull request changes the pallet Events variants from tuples to structs with named fields.

Part implementation for #9903 and #9993.

Pallets With Changes

  • uniques
  • utility
  • vesting

Polkadot address: 12dVgRw6ntsMQiETAoCBzu6MA7LPWagvU4Y1rH9jGPDHQvPX

@cla-bot-2021
Copy link

cla-bot-2021 bot commented Nov 4, 2021

User @ayevbeosa, please sign the CLA here.

@ayevbeosa ayevbeosa requested a review from athei as a code owner November 4, 2021 19:29
@athei athei added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed. labels Nov 5, 2021
Copy link
Member

@athei athei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comments need to be updated.

utils/frame/remote-externalities/Cargo.toml Outdated Show resolved Hide resolved
frame/uniques/src/lib.rs Outdated Show resolved Hide resolved
@athei
Copy link
Member

athei commented Nov 15, 2021

@ayevbeosa You need to merge master and resolve conflict before we can merge.

@ayevbeosa
Copy link
Contributor Author

@ayevbeosa You need to merge master and resolve conflict before we can merge.

Doing so now.

… ayevbeosa-struct-variants-with-named-fields

� Conflicts:
�	Cargo.lock
@athei
Copy link
Member

athei commented Nov 15, 2021

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 7ebab91 into paritytech:master Nov 15, 2021
@athei
Copy link
Member

athei commented Nov 15, 2021

/tip small

@substrate-tip-bot
Copy link

Please fix the following problems before calling the tip bot again:

  • Contributor did not properly post their Polkadot or Kusama address. Make sure the pull request has: "{network} address: {address}".

@athei
Copy link
Member

athei commented Nov 15, 2021

@ayevbeosa You need to provide an address in your opening post to receive a tip. Like here.

@ayevbeosa
Copy link
Contributor Author

@ayevbeosa You need to provide an address in your opening post to receive a tip. Like here.

This has been done. Thanks for the tip.

@athei
Copy link
Member

athei commented Nov 15, 2021

/tip small

@substrate-tip-bot
Copy link

A small tip was successfully submitted for ayevbeosa (12dVgRw6ntsMQiETAoCBzu6MA7LPWagvU4Y1rH9jGPDHQvPX on polkadot).

https://polkadot.js.org/apps/#/treasury/tips

grishasobol pushed a commit to gear-tech/substrate that referenced this pull request Mar 28, 2022
* Changed named fields to struct variants

* Fixed errors.

* Made adjustments as per `cargo +nightly fmt`.

* Update frame/uniques/src/lib.rs

Co-authored-by: Alexander Theißen <[email protected]>

* Removed redundant comments for structs.

* Moved frame-support to dev dependencies

Co-authored-by: Alexander Theißen <[email protected]>
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
* Changed named fields to struct variants

* Fixed errors.

* Made adjustments as per `cargo +nightly fmt`.

* Update frame/uniques/src/lib.rs

Co-authored-by: Alexander Theißen <[email protected]>

* Removed redundant comments for structs.

* Moved frame-support to dev dependencies

Co-authored-by: Alexander Theißen <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants