stackchain/stackchain-dashboard
1
0
Fork 0
Code Issues 9 Pull Requests 1 Actions Packages Projects Releases 713 Wiki Activity

Resume mobile pull review progress across interruptions #125

New Issue
Closed
opened 2026-08-06 17:49:07 +00:00 by rockachopa · 0 comments
rockachopa commented 2026-08-06 17:49:07 +00:00
Member
Copy Link

Problem

The mobile review inspector previews changed files but loses the reviewer’s place whenever the sheet closes or the page reloads. Reviewers must remember which files they checked, and saved state cannot safely distinguish new commits because the review payload omits the pull head SHA.

Slice

Add explicit, local, commit-aware file review progress: show reviewed count, let users mark files reviewed, jump to the next unreviewed file, restore progress for the same pull/head SHA, and reset safely when commits change. Keep final review submission in authenticated Gitea.

Acceptance tests

  • A new three-file review shows 0 of 3 files reviewed.
  • Expanding a diff does not mark it; explicit Mark reviewed advances progress and the next unreviewed file.
  • Closing/reopening or reloading restores progress for the same repository, pull number, and head SHA.
  • A changed head SHA starts fresh progress.
  • Completion preserves the Open in Gitea handoff and adds no write endpoint.
  • Controls are keyboard-operable, use 44px touch targets, and do not cause 320px horizontal overflow.
  • Existing review authorization and read-only API behavior remain intact.
## Problem The mobile review inspector previews changed files but loses the reviewer’s place whenever the sheet closes or the page reloads. Reviewers must remember which files they checked, and saved state cannot safely distinguish new commits because the review payload omits the pull head SHA. ## Slice Add explicit, local, commit-aware file review progress: show reviewed count, let users mark files reviewed, jump to the next unreviewed file, restore progress for the same pull/head SHA, and reset safely when commits change. Keep final review submission in authenticated Gitea. ## Acceptance tests - A new three-file review shows `0 of 3 files reviewed`. - Expanding a diff does not mark it; explicit **Mark reviewed** advances progress and the next unreviewed file. - Closing/reopening or reloading restores progress for the same repository, pull number, and head SHA. - A changed head SHA starts fresh progress. - Completion preserves the **Open in Gitea** handoff and adds no write endpoint. - Controls are keyboard-operable, use 44px touch targets, and do not cause 320px horizontal overflow. - Existing review authorization and read-only API behavior remain intact.
timmy was assigned by rockachopa 2026-08-06 17:49:07 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels   
P0

Drop everything

  
P1

High priority

  
P2

Medium priority

  
agent:nightprowl

Assignments routed to the LongCat NightProwl evaluation worker

  
backend

Backend service

  
ci

Continuous integration / actions

  
frontend

Frontend / UI

  
meta

Meta / planning

  
pages

Gitea Pages

  
template

Repo template / standards

No Label
P0
P1
P2
agent:nightprowl
backend
ci
frontend
meta
pages
template
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
timmy
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#125
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?