Skip to content

fix: address Section 5 case detail UAT issues#43

Merged
Taleef7 merged 1 commit into
mainfrom
fix/section-5-case-detail
May 20, 2026
Merged

fix: address Section 5 case detail UAT issues#43
Taleef7 merged 1 commit into
mainfrom
fix/section-5-case-detail

Conversation

@Taleef7

@Taleef7 Taleef7 commented May 20, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds an escalation confirmation dialog on case detail before /escalate is called.
  • Fixes outreach delivery status so the initial send returns and displays the real delivery result (SIMULATED on the demo stack) without requiring manual "Mark sent".
  • Adds a shared JSON/HTML audit packet export control for case, run, and measure-version packets.
  • Corrects Section 5 and Section 11 walkthrough-guide inaccuracies and adds the dated journal entry.

Verification

  • backend/gradlew.bat test --tests com.workwell.caseflow.CaseUpsertIntegrationTest
  • backend/gradlew.bat --no-daemon test --tests com.workwell.web.CaseControllerTest --tests com.workwell.web.AuditorControllerTest
  • frontend/corepack pnpm lint (passes with existing test/mocks/next-font.ts warning)
  • frontend/corepack pnpm test (40/40)
  • frontend/corepack pnpm build
  • Playwright local confirmation with mocked API data: escalation waits for confirmation, outreach badge updates to Simulated, and case/run/measure packet exports honor format=html.

Note

  • Attempted full backend/gradlew.bat test; it did not return before the 15-minute timeout, so the issue-specific backend checks above are the fresh passing evidence for this branch.

Closes #28

Copilot AI review requested due to automatic review settings May 20, 2026 18:47
@vercel

vercel Bot commented May 20, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
workwell-measure-studio Ready Ready Preview, Comment May 20, 2026 6:48pm

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@Taleef7 Taleef7 self-assigned this May 20, 2026

@Taleef7 Taleef7 left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Reviewed the Section 5 diff against the UAT items. I did not find any blocking code issues in this pass. The escalation confirmation, simulated delivery status propagation, shared audit-packet format selector, and guide updates look aligned with the stated acceptance criteria. I would still wait for the pending backend CI job to finish before merging.

@Taleef7 Taleef7 merged commit b547190 into main May 20, 2026
12 of 13 checks passed
@Taleef7 Taleef7 deleted the fix/section-5-case-detail branch May 20, 2026 19:40
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.

UAT Fix: Section 5 — Case Detail: outreach badge stuck NOT SENT, escalate needs confirmation, audit packet format selector

2 participants