Supply archive: mark an unfrozen past run as still open (Maestro 29b629aa) #200

Member

Maestro task 29b629aa-1ad9-48a3-be75-0380b645d4fe.

The archive marks a past unfrozen run as still open and links to the editable Supply run route instead of presenting it as a frozen read-only post; frozen posts stay read-only. No schema change. Note for the owner: the wording 'Still open' is new and not in the design handoff.

Maestro task 29b629aa-1ad9-48a3-be75-0380b645d4fe. The archive marks a past unfrozen run as still open and links to the editable Supply run route instead of presenting it as a frozen read-only post; frozen posts stay read-only. No schema change. Note for the owner: the wording 'Still open' is new and not in the design handoff.
orchestrator-agent added 1 commit 2026-10-11 09:36:31 -04:00
Mark unfrozen Supply archive posts and open them in the Supply run
CI / ci (push) Successful in 8m9s
CI / ci (pull_request) Successful in 8m7s
c2a63192a6
The Supply archive presented every post as a frozen, read-only list, even a
past week's Supply run that week_rollover had not frozen and that the
Supply run route could still change. Each archive post now carries frozen;
a post not yet frozen is marked (Current, or Still open for a past week)
and links to the Supply run, where it can still change. Only frozen posts
open read-only in the archive.

Maestro task 29b629aa-1ad9-48a3-be75-0380b645d4fe

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
overseer added 1 commit 2026-10-11 10:35:33 -04:00
overseer added 1 commit 2026-10-11 16:14:55 -04:00
Merge the feature branch into task/supply-archive-unfrozen-run (Maestro 29b629aa refresh)
CI / ci (push) Successful in 9m26s
CI / ci (pull_request) Successful in 8m32s
5637bf872e
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
review_agent requested changes 2026-10-11 16:16:16 -04:00
Dismissed
review_agent left a comment
Member

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-11 16:34:05 -04:00
Supply archive: open an unfrozen ?week= visit in the editable Supply run (Maestro 29b629aa)
CI / ci (push) Successful in 8m32s
CI / ci (pull_request) Successful in 8m24s
3a4991cc8e
Review remediation for task 29b629aa-1ad9-48a3-be75-0380b645d4fe.

Finding 1 (medium): /plan/shopping/archive?week=<week> forced the list
read-only whatever its status, so a saved link to a Supply run not yet
frozen showed it tagged Archived without checkboxes. Read-only, a run
not yet frozen now opens the editable Supply run screen instead
(replace), and only a frozen run is tagged Archived. The desktop test
that pinned the old read-only behaviour now expects the editable
routing, and a new test covers a past week not yet frozen.

Finding 2 (low): the card link uses the exported supplyRunScreen()
instead of a duplicate supplyRun() helper.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
overseer added 1 commit 2026-10-11 17:04:06 -04:00
review_agent left a comment
Member

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 cfc229049f into feature/7383c2cb-3179-4b45-b558-bf1bc1833fad-launch-readiness 2026-10-11 17:29:09 -04:00
Sign in to join this conversation.
No Reviewers
No labels
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Maniacs_Mansion/MealQuest#200