Review saved pull request diffs and draft feedback offline #356

Merged
timmy merged 1 commits from timmy/355-offline-review-drafts into main 2026-08-09 00:22:08 +00:00
Owner

What changed

  • warm bounded, allowlisted requested-review details with Today offline readiness
  • open saved diffs, prior review context, and SHA-scoped drafting in the mobile review sheet
  • disable offline submission and revalidate the current review head on reconnect
  • ship the changed browser assets atomically in shell cache v44

Verification

  • strict RED/GREEN focused tests for review storage and offline opening
  • python3 -m pytest tests/ -q — 663 passed
  • Chromium 390×844 rendered flow — full-width sheet, diff/inline draft persisted, submit disabled with reconnect guidance, zero page errors

Closes #355

## What changed - warm bounded, allowlisted requested-review details with Today offline readiness - open saved diffs, prior review context, and SHA-scoped drafting in the mobile review sheet - disable offline submission and revalidate the current review head on reconnect - ship the changed browser assets atomically in shell cache v44 ## Verification - strict RED/GREEN focused tests for review storage and offline opening - `python3 -m pytest tests/ -q` — 663 passed - Chromium 390×844 rendered flow — full-width sheet, diff/inline draft persisted, submit disabled with reconnect guidance, zero page errors Closes #355
timmy added 1 commit 2026-08-09 00:20:57 +00:00
feat: draft requested reviews offline (#355)
All checks were successful
CI / lint (pull_request) Successful in 37s
CI / build-release (pull_request) Successful in 5s
CI / release-candidate (pull_request) Has been skipped
229d112297
timmy merged commit 3384e56cb8 into main 2026-08-09 00:22:08 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: stackchain/stackchain-dashboard#356
No description provided.