Repository navigation
[AGENT] Add flexible event lineups and time zone defaults - #366
Conversation
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: 3ac6a98df7
ℹ️ 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".
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
PR verification reportAll configured preview and verification checks passed.
Changed visual baselines (110)
Updated from Baseline Checks run 38490011615, attempt 1 for |
|
[AGENT] @greptileai review Please refresh the review for d5b376f. The reported anchor and rejected-duration defects have fixes and desktop/mobile regressions; all handled threads have dispositions. |
|
[AGENT] @greptileai review Please refresh for 679b87a. The optional end-clearing regression is fixed, with canonical/legacy two-save persistence and desktop/mobile coverage. The handled thread is resolved and the scoped review is clean. |
|
[AGENT] Claude ReviewClaude review for commit This is not a current review or approval. |
|
[AGENT] @greptileai review Please refresh for c779488. The PR now targets main after #365 merged. The main integration passed an independent scoped review; the scheduling files are preserved and the shared schema retains both sets of fields. |
|
[AGENT] Projection clarification for the Claude review: the new untimed provenance fields are omitted from public/editor/operator lineup entries. After retiming, existing public slots[].source still includes type, label and sanitized URL. Notes remain private. No public DTO field was added. The checkpoint's storage-verification section records that distinction. |
[AGENT] Makes event lineups editable after initial generation. Contributors and staff can insert a timed slot between performers, change its duration, or remove it while later timed slots move by the elapsed-time difference. Explicit event ends stay fixed, with a conflict and an action to extend them.
Adds a compact searchable time zone picker with common aliases and a private account default. Review has one prominent completion action using the existing publish/save commands. Reset confirms before clearing the lineup and keeps event details and artwork.
The older staff update path now carries validated stored-slot identities so timing edits preserve hidden performer links and unchanged unavailable streams. Canonical timing edits preserve unchanged performer confirmation and source attribution, including timed/untimed conversions. Legacy creation still requires a start time, and legacy untimed performers remain in their existing group.
Validation includes Linux desktop/mobile screenshots and actual backend regressions for protected associations, unavailable streams and forged slot mappings. Final results are recorded in the verification checkpoint.