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

Make assigned pull request sheets read-first with on-demand review #218

Merged
timmy merged 1 commits from timmy/217-read-first-pull-sheet into main 2026-08-07 18:01:16 +00:00
Conversation 0 Commits 1 Files Changed 6 +251 -26
timmy commented 2026-08-07 18:00:24 +00:00
Owner
Copy Link

Closes #217

What changed

  • split lightweight pull reading/conversation data from review and merge data
  • render description, complete conversation, and draft-safe reply before a collapsed Review & merge disclosure
  • lazily load changed files, diff previews, CI, and merge readiness with single-flight head-scoped caching and scoped retry
  • preserve all existing reviewed-file and server-side merge gates

Verification

  • RED: 4 focused behavioral tests failed for eager review requests, missing review endpoint/controller, and missing collapsed hierarchy
  • GREEN: python3 -m pytest tests/ -q → 341 passed
  • mobile render: Chromium at 320x568 and 390x844 → no page/panel overflow; read-first order; 44px review and merge controls
Closes #217 ## What changed - split lightweight pull reading/conversation data from review and merge data - render description, complete conversation, and draft-safe reply before a collapsed Review & merge disclosure - lazily load changed files, diff previews, CI, and merge readiness with single-flight head-scoped caching and scoped retry - preserve all existing reviewed-file and server-side merge gates ## Verification - RED: 4 focused behavioral tests failed for eager review requests, missing review endpoint/controller, and missing collapsed hierarchy - GREEN: `python3 -m pytest tests/ -q` → 341 passed - mobile render: Chromium at 320x568 and 390x844 → no page/panel overflow; read-first order; 44px review and merge controls
timmy added 1 commit 2026-08-07 18:00:25 +00:00
feat: make assigned pull sheets read-first (#217)
All checks were successful
CI / lint (pull_request) Successful in 20s
Details
CI / build-frontend (pull_request) Successful in 5s
Details
174dac05b9
timmy merged commit 3824ff180a into main 2026-08-07 18:01:16 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
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
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#218
No description provided.
Delete Branch "timmy/217-read-first-pull-sheet"

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?