Skip to content

Update full-day date range defaults#69

Merged
felixgollnhuber merged 2 commits intomainfrom
codex/change-default-date-range-for-full-day-events
Apr 27, 2026
Merged

Update full-day date range defaults#69
felixgollnhuber merged 2 commits intomainfrom
codex/change-default-date-range-for-full-day-events

Conversation

@felixgollnhuber
Copy link
Copy Markdown
Owner

Summary

  • Set the default date range for full-day events to start today and extend one month ahead.
  • Keep the existing time-grid default range unchanged.
  • Add coverage for the full-day UI label and submitted date span.

Testing

  • Added and updated unit/UI coverage in src/components/create-event-form.test.tsx.
  • Verified the focused component test suite locally.

Copilot AI review requested due to automatic review settings April 27, 2026 21:36
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7f951d3031

ℹ️ 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".

Comment thread src/components/create-event-form.tsx Outdated
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Create Event form so full-day events default to a “today → one month ahead” date range while keeping the existing time-grid defaults, and adds UI/submission assertions to cover the new behavior.

Changes:

  • Added full-day default date range logic (today through +1 month) while preserving the time-grid default range.
  • Updated event type selection to reset date range only when the user is still on the current default.
  • Expanded CreateEventForm tests to assert the full-day range label and submitted date span.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/components/create-event-form.tsx Introduces new default date range helpers and updates event-type switching behavior to apply appropriate defaults.
src/components/create-event-form.test.tsx Adds assertions for full-day date range label and submitted date boundaries.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/components/create-event-form.tsx Outdated
Comment thread src/components/create-event-form.tsx Outdated
@felixgollnhuber felixgollnhuber merged commit 565f6d0 into main Apr 27, 2026
2 checks passed
@felixgollnhuber felixgollnhuber deleted the codex/change-default-date-range-for-full-day-events branch April 27, 2026 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants