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

fix: await upsert-d-parameter and upsert-permissioned-candidates transaction outputs being present #360

Merged
merged 1 commit into from
Jan 10, 2025

Conversation

LGLO
Copy link
Contributor

@LGLO LGLO commented Jan 9, 2025

Description

Not awaiting for the output being present made it possible that the following transactions fail, because they query wallet before previous transaction being observable but try to submit it after. Happened during testing wizards.

Checklist

  • Commit sequence broadly makes sense and commits have useful messages.
  • New tests are added if needed and existing tests are updated.
  • Relevant logging and metrics added
  • CI passes. See note on CI.
  • Any changes are noted in the changelog.md for affected crate
  • Self-reviewed the diff

@LGLO LGLO force-pushed the ETCM-8949-fix-not-awaiting-for-tx-submission branch from 57a3e09 to aad3a12 Compare January 10, 2025 06:52
@LGLO LGLO enabled auto-merge (squash) January 10, 2025 06:52
@LGLO LGLO merged commit c25a88b into master Jan 10, 2025
14 checks passed
@LGLO LGLO deleted the ETCM-8949-fix-not-awaiting-for-tx-submission branch January 10, 2025 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants