Maniacs_Mansion/MealQuest
11
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Keep it in rotation card (Feature 21) #179

Merged
approver_agent merged 2 commits from task/keep-it-in-rotation-card_4647eadf-4545-4d3f-849f-ffa5e9fcbd1b into feature/446472d5-2a84-4aa2-bb37-fdc80815615c-home-dashboard 2026-10-10 17:13:20 -04:00
Conversation 2 Commits 2 Files Changed 10
+598 -8
dev_agent commented 2026-10-10 15:55:57 -04:00
Member
Copy Link
Copy Source

Maestro task 4647eadf-4545-4d3f-849f-ffa5e9fcbd1b.

Home shows the oldest pending rotation review above the tiles, with Keep it / Not for us / We didn't make it. Answers call POST /api/v1/dinners/{id}/rotation-review optimistically, show the confirmation, and roll back with an ErrorBanner on failure. Nothing is decided automatically.

Head: be8a658742, based on feature head 837da07f3e.

Developer reports web typecheck, test and build passed. CI has not run yet.

Maestro task 4647eadf-4545-4d3f-849f-ffa5e9fcbd1b. Home shows the oldest pending rotation review above the tiles, with Keep it / Not for us / We didn't make it. Answers call POST /api/v1/dinners/{id}/rotation-review optimistically, show the confirmation, and roll back with an ErrorBanner on failure. Nothing is decided automatically. Head: be8a6587425968c66a53054854f4e968f528e33e, based on feature head 837da07f3e392c274a3d2b6c4f6681e47c1b4f1a. Developer reports web typecheck, test and build passed. CI has not run yet.
dev_agent added 1 commit 2026-10-10 15:55:58 -04:00
Add the Keep it in rotation card to Home (Feature 21)
CI / ci (push) Successful in 8m20s
Details
CI / ci (pull_request) Successful in 8m1s
Details
be8a658742 
Home shows the oldest pending Rotation check (GET /api/v1/rotation-reviews)
above the tiles: washed photo, "Last night: {dinner}", "Keep it in
rotation?" and Keep it / Not for us / We didn't make it. An answer posts
to /api/v1/dinners/{id}/rotation-review optimistically, confirms it and
reveals the next check; a failure puts the check back with an ErrorBanner.
Nothing is decided automatically.

Task: 4647eadf-4545-4d3f-849f-ffa5e9fcbd1b

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
review_agent requested changes 2026-10-10 16:28:38 -04:00
Dismissed
review_agent left a comment
Member
Copy Link
Copy Source

Independent Codex gpt-6-sol review: CHANGES_REQUIRED for this exact commit.

Independent Codex gpt-6-sol review: CHANGES_REQUIRED for this exact commit.
overseer added 1 commit 2026-10-10 16:31:08 -04:00
Refetch Rotation checks when a notification focuses Home already open (Feature 21)
CI / ci (push) Successful in 8m48s
Details
CI / ci (pull_request) Successful in 9m34s
Details
1cc5b4f6ab 
A notification tap that focuses a window already at its url does not
navigate, so Home does not remount and a recently read empty list stays
within the query client's staleTime. The service worker now posts that
window NotificationOpened, and useRotationReviews refetches on it, so the
Keep it in rotation card shows without a remount.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
review_agent left a comment
Member
Copy Link
Copy Source

Independent Codex gpt-6-sol review: APPROVED for this exact commit.

Independent Codex gpt-6-sol review: APPROVED for this exact commit.
approver_agent merged commit 62f4f9eece into feature/446472d5-2a84-4aa2-bb37-fdc80815615c-home-dashboard 2026-10-10 17:13:20 -04:00
Sign in to join this conversation.
Reviewers
No Reviewers
review_agent
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
approver_agent (The Approver) chatgpt claude-ai Claude-Code codex (Codex) dev_agent grok opencode orchestrator-agent overseer (Joshua) review_agent (The Reviewer)
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Maniacs_Mansion/MealQuest#179