Skip to content

Comments

Bookings Table now defaults to the "Today" timeframe#1225

Open
interfinityOfficial wants to merge 2 commits intomainfrom
1085-bookings-table-default-to-today
Open

Bookings Table now defaults to the "Today" timeframe#1225
interfinityOfficial wants to merge 2 commits intomainfrom
1085-bookings-table-default-to-today

Conversation

@interfinityOfficial
Copy link
Collaborator

Checklist

  • I checked for existing implementations and confirmed there is no duplication
  • I tested this feature locally
  • I had Copilot review the PR and incorporated feedback (or explained why not)
  • I confirmed there are no conflicts
  • I confirmed my PR passed all tests
  • I added or updated unit tests (or explained why not)
  • I attached screenshots or a video demonstrating the feature
  • I requested a code review from at least one other teammate

Screenshots / Video

Screenshot 2026-02-18 at 4 23 16 PM

@rlho
Copy link
Collaborator

rlho commented Feb 23, 2026

@interfinityOfficial
Can you assign Copilot as a reviewer before assigning me as a reviewer?

Copy link
Contributor

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

This PR updates the default timeframe filter for the Bookings table from "All Future" to "Today", providing a more focused initial view of bookings for the current day. This aligns with user expectations and makes the interface more immediately useful for day-to-day operations.

Changes:

  • Modified the default selectedDateRange state initialization from "All Future" to "Today" in the Bookings component

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

@rlho
Copy link
Collaborator

rlho commented Feb 23, 2026

@interfinityOfficial
You checked “Add unit tests” in the checklist, but I can’t find them. Can you add them?

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.

As a PA I want the bookings table to default to the "Today" timeframe, not the "All Future" timeframe.

2 participants