Skip to content
This repository was archived by the owner on Jul 31, 2026. It is now read-only.

fix: allow pending transactions to be updated when confirmed - #194

Merged
ulissesferreira merged 1 commit into
mainfrom
NEB-473/fix-pending-transaction-status-update
Feb 5, 2026
Merged

fix: allow pending transactions to be updated when confirmed#194
ulissesferreira merged 1 commit into
mainfrom
NEB-473/fix-pending-transaction-status-update

Conversation

@ulissesferreira

Copy link
Copy Markdown
Contributor

The incremental sync was filtering out all transactions whose IDs already existed in state, including pending transactions. This meant pending transactions could never be updated to confirmed status.

Changed to only skip transactions that are already confirmed/failed, allowing pending transactions to be re-fetched and updated.

@ulissesferreira
ulissesferreira requested a review from a team as a code owner February 5, 2026 17:02
The incremental sync was filtering out all transactions whose IDs
already existed in state, including pending transactions. This meant
pending transactions could never be updated to confirmed status.

Changed to only skip transactions that are already confirmed/failed,
allowing pending transactions to be re-fetched and updated.
@ulissesferreira
ulissesferreira force-pushed the NEB-473/fix-pending-transaction-status-update branch from f5e898b to d41aa44 Compare February 5, 2026 17:16

@mikesposito mikesposito left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@ulissesferreira
ulissesferreira merged commit 0dcda94 into main Feb 5, 2026
7 of 8 checks passed
@ulissesferreira
ulissesferreira deleted the NEB-473/fix-pending-transaction-status-update branch February 5, 2026 17:24
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.

2 participants