Repository navigation
chore: remove retired worker calendar health runtime - #202
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0b8e12c088
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Deployment verification: main 2660815 reached READY at https://piccnewyork.org in deployment dpl_8gwTQiQa13PtZHPifqgkAgPPziqx and remains included in the subsequent dashboard release. Local mobile browser proof confirmed the retired calendar health tile is absent and the retired authenticated endpoint returns 410. Authenticated production interaction has not been exercised in this session. |
|
Authenticated production browser check now confirms the expanded Home sync panel contains retailer and order sources, with the retired calendar source absent. No app records were modified. |
This branch was successfully deployed
1 participant
Home was querying retired worker/calendar tables and reporting obsolete availability-sync health. Remove that query/tile and unused service; the old endpoint now returns 410 after its existing authentication and role checks. Active retailer and order freshness remain available in the Home disclosure.
Closes #201. Prerequisite for #143; no schema/data/history changes.
Validation: RED regression first, then npm run verify passed (57 test files / 250 tests, lint, types, Prisma validation, build). Mobile browser opened the Home sync disclosure and verified retailer/order status, no retired tile, and HTTP 410 from the legacy endpoint. Local seeded database only; signed-in production behavior remains separately unverified.
Owned paths: Home page, legacy sync-health endpoint/service, focused endpoint test, SESSION.md. #195 overlaps Home and must rebase while preserving its org guard. No auth policy changes, new permissions, or production writes. Fast-lane source cleanup.