docs: fix checkout status lifecycle diagram to match spec prose - #672
Open
SumuduLansakara wants to merge 1 commit into
Open
docs: fix checkout status lifecycle diagram to match spec prose#672SumuduLansakara wants to merge 1 commit into
SumuduLansakara wants to merge 1 commit into
Conversation
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
The Checkout Status Lifecycle diagram only showed a one-way path from ready_for_complete into complete_in_progress, with no way back to incomplete and no path to requires_escalation. The normative prose elsewhere in this file describes both: - Complete Checkout's response section states that any other status retains its core status lifecycle semantics, explicitly naming incomplete (for a recoverable change) and requires_escalation (for Buyer handoff) as valid outcomes alongside completed and complete_in_progress. - ready_for_complete can regress to incomplete via Update Checkout as well (business rules can reintroduce a missing requirement). Update the diagram to show: - ready_for_complete -> requires_escalation, labeled with the Complete Checkout escalation path. - ready_for_complete <-> incomplete as two directional arrows: the existing all info collected transition down, and a new Update Checkout, or Complete Checkout (recoverable) transition back up. Add a note documenting the two transitions still omitted from the diagram for readability (complete_in_progress resolving via Buyer handoff or via the cancellation race described in Accepted completion), so the diagram's simplifications are explicit rather than silent. No normative or schema changes; documentation only.
SumuduLansakara
force-pushed
the
main-fork
branch
from
August 2, 2026 08:46
262ecbc to
d34a01a
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
The Checkout Status Lifecycle diagram only showed a one-way path from
ready_for_completeintocomplete_in_progress, with no way back toincompleteand no path torequires_escalation. The normative proseelsewhere in this file describes both:
retains its core status lifecycle semantics, explicitly naming
incomplete("for a recoverable change") andrequires_escalation("for Buyer handoff") as valid outcomes alongside
completedandcomplete_in_progress.ready_for_completecan regress toincompletevia Update Checkoutas well (business rules can reintroduce a missing requirement).
Update the diagram to show:
ready_for_complete -> requires_escalation, labeled with theComplete Checkout escalation path.
ready_for_complete <-> incompleteas two directional arrows: theexisting "all info collected" transition down, and a new "Update
Checkout, or Complete Checkout (recoverable)" transition back up.
Add a note documenting the two transitions still omitted from the
diagram for readability (
complete_in_progressresolving via Buyerhandoff or via the cancellation race described in Accepted
completion), so the diagram's simplifications are explicit rather
than silent.
No normative or schema changes; documentation only.
Category (Required)
ucp-schematool (resolver, linter, validator). (Requires Maintainer approval)Related Issues
None filed — found while reading the spec, opening directly as a docs fix.
Checklist
!for breaking changes).Screenshots / Logs (if applicable)
Before:
After: see the updated diagram in
docs/specification/checkout.md.