Repository navigation
Show live staff case progress and announce real acceptance - #41
Merged
Merged
Conversation
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.
Customer failure prevented
The staff card stays at Needs a human after a real acceptance, and the customer never sees who picked up the case. If the acceptance notice cannot be confirmed, staff must receive an actionable private instruction rather than a log-only warning.
Change
Phase 3 commits
551e48e: merge released main8321439into live case status (two-parent merge commit).e5cff9e: explicit private follow-up instruction and notification/boot/timer/stale-card regressions.Validation
Production and release gates
Report 1 was graded PASS. The six separately approved contributor fixes are now live as main
8321439, deployment5b25759b-d01e-4645-ade1-c869a482b323; Report 2 is awaiting grading. The actual combined production suite was 638 tests and offline 8/8.This PR remains draft: no PR #41 merge/deploy, Railway variable changes, protected-file reads or protected rerun. The latest staged order is Phase 2 → reviewed #41 → routing comparison → reviewed/private-preflight #28 → reviewed #29 → reviewed #30. #28's audience validation is not claimed as integrated here; no competing validator is added. Required wording/release gates and Discord accept/double-accept/start/wait/refresh/restart/close smoke remain before enablement. Existing cards can be adopted through an authorized Refresh or Accept action. A crash between the durable notice claim and Discord delivery may leave that optional notice unconfirmed; staff follow-up remains the fallback, not an exactly-once guarantee.
#28–#30 are unchanged by this update. N12 credential rotation remains deferred by Aryan (Oct 9), accepted risk.