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

Carry mobile review feedback into the authenticated Gitea handoff #129

New Issue
Closed
opened 2026-08-06 18:49:44 +00:00 by timmy · 0 comments
timmy commented 2026-08-06 18:49:44 +00:00
Owner
Copy Link

User problem

The mobile requested-review sheet lets reviewers inspect and mark files reviewed, but it cannot capture per-file observations or carry an overall decision into the existing authenticated Gitea handoff. Reviewers must reconstruct feedback from memory after leaving the dashboard.

Evidence

  • frontend/index.html exposes changed files, progress, and an "Open in Gitea" link but no feedback draft controls.
  • frontend/review-sheet.js persists only reviewed filenames by repository, PR, and head SHA.
  • Historical issue/PR title review through #128 and recent origin/main history show no shipped review-note or decision-draft workflow.

Acceptance criteria

  • Persist per-file notes, overall summary, and intended decision for the exact repository/PR/head SHA.
  • Restore the draft after closing/reloading; a new head SHA starts a fresh draft.
  • Copy structured Markdown containing non-empty notes in file order and the summary/decision before opening the authenticated Gitea PR.
  • Clipboard failure keeps feedback available in a selectable fallback and does not navigate away.
  • Controls remain usable on a 320px-wide phone with 44px touch targets.
  • Keep the dashboard read-only: introduce no Gitea mutation endpoint.
  • Add behavioral tests and keep the complete suite green.
## User problem The mobile requested-review sheet lets reviewers inspect and mark files reviewed, but it cannot capture per-file observations or carry an overall decision into the existing authenticated Gitea handoff. Reviewers must reconstruct feedback from memory after leaving the dashboard. ## Evidence - `frontend/index.html` exposes changed files, progress, and an "Open in Gitea" link but no feedback draft controls. - `frontend/review-sheet.js` persists only reviewed filenames by repository, PR, and head SHA. - Historical issue/PR title review through #128 and recent `origin/main` history show no shipped review-note or decision-draft workflow. ## Acceptance criteria - Persist per-file notes, overall summary, and intended decision for the exact repository/PR/head SHA. - Restore the draft after closing/reloading; a new head SHA starts a fresh draft. - Copy structured Markdown containing non-empty notes in file order and the summary/decision before opening the authenticated Gitea PR. - Clipboard failure keeps feedback available in a selectable fallback and does not navigate away. - Controls remain usable on a 320px-wide phone with 44px touch targets. - Keep the dashboard read-only: introduce no Gitea mutation endpoint. - Add behavioral tests and keep the complete suite green.
timmy self-assigned this 2026-08-06 18:49:44 +00:00
timmy closed this issue 2026-08-06 18:57:14 +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#129
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?