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

Add submit_and_watch_extrinsic_until_success method #373

Closed
wants to merge 8 commits into from

Conversation

haerdib
Copy link
Contributor

@haerdib haerdib commented Dec 14, 2022

Allows to watch an extrinsic and automatically checks if the extrinsic was successful or not, by retrieving and checking the associated events.

❗ Breaks submit_and_watch_extrinsic_until interface: The return value has been changed to ExtrinsicReport which includes the extrinsic hash, block hash, transaction status and, possibly, the associated events.

closes #288

@haerdib haerdib self-assigned this Dec 14, 2022
@haerdib haerdib added E2-breaksapi F8-newfeature Introduces a new feature labels Dec 14, 2022
@haerdib haerdib force-pushed the bh/277-test-for-xt-success branch from 018e084 to 9334f13 Compare December 16, 2022 10:53
@haerdib haerdib force-pushed the bh/277-test-for-xt-success branch from 9711af9 to c3a6387 Compare December 16, 2022 12:36
@haerdib
Copy link
Contributor Author

haerdib commented Dec 20, 2022

Replaced by #385

@haerdib haerdib closed this Dec 20, 2022
@haerdib haerdib deleted the bh/277-test-for-xt-success branch December 23, 2022 15:48
@haerdib haerdib restored the bh/277-test-for-xt-success branch December 23, 2022 15:48
@haerdib haerdib deleted the bh/277-test-for-xt-success branch January 3, 2023 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E2-breaksapi F8-newfeature Introduces a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

send_extrinsic(xt, XtStatus::Finalized) does not report error upon xt failure
1 participant