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.

Sign up for GitHub

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

Jump to bottom

chore: update `Sudid` destructure docs #465

Open

peetzweg wants to merge 1 commit into polkadot-js:master from peetzweg:pz/sudid-destructuring

peetzweg commented Feb 16, 2024

The docs destructure an array element form the Sudid event. This is no longer available. It is now a named field called sudo_result.

Came up in this substack question:
https://substrate.stackexchange.com/questions/10981/property-iserror-does-not-exist-on-type-codec-when-showing-sudo-events

And here is the actual Event defintion:
https://github.com/paritytech/polkadot-sdk/blob/e2caa813bfd2973e7a7a74aa5292433397dbc787/substrate/frame/sudo/src/lib.rs#L302-L306


          update Sudid destructure docs

216bf39

TarikGul approved these changes

View reviewed changes

Member

TarikGul left a comment

Makes total sense. Thanks for the update

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet