Skip to content

Enable resetDownstreamState from the build step when changing fields - #2182

Open
jeesunikim wants to merge 3 commits into
mainfrom
build-flow-reset-on-edit
Open

Enable resetDownstreamState from the build step when changing fields#2182
jeesunikim wants to merge 3 commits into
mainfrom
build-flow-reset-on-edit

Conversation

@jeesunikim

Copy link
Copy Markdown
Contributor
  • When a user edits fields on the build step after having already progressed past it with a built transaction, reset the xdr from the build step. Previously, it got reset to a new xdr with a new value once a user signed the updated tx at sign transaction step. This resets it from build transaction step.

Copilot AI balanced review requested due to automatic review settings August 20, 2026 23:45
@github-project-automation github-project-automation Bot moved this to Backlog (Not Ready) in DevX Aug 20, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Resets stale downstream transaction state when build fields change after progression.

Changes:

  • Detects built XDR changes and resets downstream steps.
  • Adds an end-to-end regression test for clearing signatures.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/app/(sidebar)/transaction/build/page.tsx Resets downstream state after XDR changes.
tests/e2e/buildFlowResetOnEdit.test.ts Tests signature invalidation after editing.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/app/(sidebar)/transaction/build/page.tsx Outdated
@stellar-jenkins-ci

Copy link
Copy Markdown

@stellar-jenkins-ci

Copy link
Copy Markdown

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

Labels

None yet

Projects

Status: Backlog (Not Ready)

Development

Successfully merging this pull request may close these issues.

2 participants