Skip to content

docs: update 3-3-approval-workflows.md - #441

Open
habib424 wants to merge 1 commit into
mainfrom
lightship/03-user-management-3-3-approval-workflows-2026-07-17
Open

docs: update 3-3-approval-workflows.md#441
habib424 wants to merge 1 commit into
mainfrom
lightship/03-user-management-3-3-approval-workflows-2026-07-17

Conversation

@habib424

Copy link
Copy Markdown
Collaborator

Drafted by Ship Ship 🚢 — confidence 🟢 high

What changed

Added a "Good to know" callout under Approval Nodes explaining that a node with no eligible active approver now causes submissions (e.g. credit notes) to be rejected with a clear error instead of hanging indefinitely, plus a corresponding Best Practices bullet about keeping approvers active.

Source PRs

  • light#7541 — feat(GREEN-1926): fail loudly on unsatisfiable approval chains

How to use this PR

  1. Review the diff against the existing article — focus on accuracy and customer voice.
  2. Merge if it reads well. Edit and push to this branch if it needs adjustment.
  3. Close without merging if the article doesn't actually need this update.

Once we have a track record of clean drafts, we'll graduate to auto-merge with a delay buffer. For now: human-in-the-loop.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant