Skip to content
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

Update Frame Pallets to use `decl_error!` #4365

Closed
shawntabrizi opened this issue Dec 11, 2019 · 4 comments
Closed

Update Frame Pallets to use `decl_error!` #4365

shawntabrizi opened this issue Dec 11, 2019 · 4 comments
Assignees
Labels
Projects
Milestone

Comments

@shawntabrizi
Copy link
Member

@shawntabrizi shawntabrizi commented Dec 11, 2019

Since #3433 has been merged in, we have had the ability to create really nice error experiences for users using Substrate Pallets.

We just need to migrate existing errors into this new format.

I expect, since this will change a bunch of modules, we will do this in a few steps. This issue will track the progress:

  • assets
  • aura
  • authority-discovery
  • authorship
  • babe
  • balances
  • collective
  • contracts
  • democracy
  • elections
  • elections-phragmen
  • evm
  • example
  • executive
  • finality-tracker
  • generic-asset
  • grandpa
  • identity
  • im-online
  • indices
  • membership
  • metadata
  • nicks
  • offences
  • randomness-collective-flip
  • scored-pool
  • session
  • society
  • staking
  • sudo
  • support
  • system
  • timestamp
  • transaction-payment
  • treasury
  • utility
@shawntabrizi shawntabrizi self-assigned this Dec 11, 2019
@shawntabrizi

This comment has been minimized.

Copy link
Member Author

@shawntabrizi shawntabrizi commented Dec 11, 2019

Sudo: #4369

@DarkEld3r

This comment has been minimized.

Copy link
Member

@DarkEld3r DarkEld3r commented Dec 11, 2019

Treasury: #4370.

As far as I can see, there are no custom errors in utility.

@DarkEld3r

This comment has been minimized.

Copy link
Member

@DarkEld3r DarkEld3r commented Dec 12, 2019

Stacking: #4387.

@shawntabrizi

This comment has been minimized.

Copy link
Member Author

@shawntabrizi shawntabrizi commented Dec 17, 2019

Balances: #4405

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Runtime
  
Done
3 participants
You can’t perform that action at this time.