Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 29, 2026

Users get stuck in "Ready for Draft" state when validation fails (e.g., missing vendor) with no visibility into errors and no path forward.

Changes

  • Error Messages FactBox: Changed Visible = false ΓåÆ Visible = HasErrorsOrWarnings so users can see validation errors
  • Finalize action visibility: Extended to show for both "Ready for Draft" and "Draft Ready" states (remains non-promoted as intended for agent-driven finalization). The underlying system automatically handles running the necessary steps (Prepare draft, then Finish draft) when finalize is invoked.
  • Test added: Added FinishDraftFromReadyForDraftStateSucceeds test to verify that invoking the finalize action from "Ready for Draft" state correctly processes the document to "Processed" state

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Fixes AB#620054

Copilot AI and others added 2 commits January 29, 2026 11:03
…eady for Draft state

Co-authored-by: Groenbech96 <17690329+Groenbech96@users.noreply.github.com>
… draft

Co-authored-by: Groenbech96 <17690329+Groenbech96@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix document draft validation issue causing user blockage Fix e-document draft UX: show error factbox and allow finalize from Ready for Draft state Jan 29, 2026
Copilot AI requested a review from Groenbech96 January 29, 2026 11:07
@Groenbech96
Copy link
Contributor

@copilot Dont add the extra reprocessing, as the underlying system will do that already.
The new condition is fine for when to show.

Please add a test that verifies that going from ready for draft, clicking finalize action, will indeed behave as expected.

…for Draft state

Co-authored-by: Groenbech96 <17690329+Groenbech96@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 29, 2026

@copilot Dont add the extra reprocessing, as the underlying system will do that already.
The new condition is fine for when to show.

...

Removed the extra reprocessing logic and added the test FinishDraftFromReadyForDraftStateSucceeds which verifies that invoking the finalize action from "Ready for Draft" state correctly processes the document to "Processed" state. Commit: 631e3e9

@github-actions github-actions bot added the AL: Apps (W1) Add-on apps for W1 label Jan 29, 2026
@Groenbech96 Groenbech96 added the Linked Issue is linked to a Azure Boards work item label Jan 30, 2026
@github-actions github-actions bot added this to the Version 28.0 milestone Jan 30, 2026
Magnus Hartvig Grønbech added 2 commits January 30, 2026 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AL: Apps (W1) Add-on apps for W1 Linked Issue is linked to a Azure Boards work item

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants