Skip to content

Conversation

@dhairyashiil
Copy link
Member

Before:

Inconsistent Behaviour:

Screen.Recording.2025-12-12.at.5.51.51.PM.mov

no validation:

Screen.Recording.2025-12-12.at.5.52.47.PM.mov

After:

Screen.Recording.2025-12-12.at.6.17.32.PM.mov

What does this PR do?

Fixes inconsistent behavior of the OOO "Show note on public booking page" feature across different booking views (month, column, weekly).

Problem

  • Month view: OOO with public notes was working correctly
  • Column view: OOO was only visible when a redirect user was set, ignoring public notes
  • Weekly view: OOO was not displayed at all without a redirect user

Solution

Applied consistent logic across all views: OOO is displayable = toUser exists OR showNotePublicly is true

Changes

Weekly View (packages/features/calendars/weeklyview/components/event/Empty.tsx)

  • Updated condition to show OOO when showNotePublicly is true (even without redirect)
  • Added notes and showNotePublicly props to OutOfOfficeInSlots component

Column View (packages/features/schedules/lib/use-schedule/useNonEmptyScheduleDays.ts)

  • Updated filtering to include days with OOO + public note

Types (packages/features/calendars/weeklyview/types/state.ts)

  • Added notes and showNotePublicly to TimeRangeExtended type

Validation (packages/features/settings/outOfOffice/CreateOrEditOutOfOfficeModal.tsx)

  • Disabled "Show note on public booking page" checkbox when notes are empty/blank
  • Auto-uncheck when notes are cleared

- Update weekly view to display OOO when showNotePublicly is true
- Update column view to include days with public notes in schedule
- Add notes and showNotePublicly props to weekly view calendar
- Disable show note publicly checkbox when notes field is empty
@graphite-app graphite-app bot added the community Created by Linear-GitHub Sync label Dec 12, 2025
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 4 files

@CarinaWolli CarinaWolli merged commit 6e2e826 into main Dec 15, 2025
81 of 89 checks passed
@CarinaWolli CarinaWolli deleted the fix/ooo-show-note-publicly-consistency branch December 15, 2025 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Created by Linear-GitHub Sync ready-for-e2e size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants