Skip to content

[Backport backport/v3.x] The recent transaction confirmation strategy now throws transaction errors from the one-shot test#803

Merged
steveluscher merged 1 commit into
backport/v3.xfrom
backport-793-to-backport/v3.x
Aug 29, 2025
Merged

[Backport backport/v3.x] The recent transaction confirmation strategy now throws transaction errors from the one-shot test#803
steveluscher merged 1 commit into
backport/v3.xfrom
backport-793-to-backport/v3.x

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Backport cfc1d92 from #793.

…rrors from the one-shot test (#793)

#### Problem

The recent signature transaction strategy does two things:

- Subscribes for changes to a signature's status
- Does a point lookup of the signature's status

It was reported that if the signature's status is an error, the one-shot lookup would not throw it.

#### Summary of Changes

Throw any transaction errors discovered during the one-shot lookup.

Fixes #711.

(cherry picked from commit cfc1d92)
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Aug 29, 2025

🦋 Changeset detected

Latest commit: 104ee48

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@steveluscher steveluscher enabled auto-merge (squash) August 29, 2025 15:56
@steveluscher steveluscher merged commit eb0a122 into backport/v3.x Aug 29, 2025
6 checks passed
@steveluscher steveluscher deleted the backport-793-to-backport/v3.x branch August 29, 2025 15:57
@github-actions github-actions Bot mentioned this pull request Aug 29, 2025
@github-actions
Copy link
Copy Markdown
Contributor Author

Because there has been no activity on this PR for 14 days since it was merged, it has been automatically locked. Please open a new issue if it requires a follow up.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant