Skip to content

fix: allow deleting first time slot in availability schedule (#26448)#27569

Merged
sahitya-chandra merged 3 commits intocalcom:mainfrom
abhayymishraa:fix/26448-allow-delete-first-time-slot
Feb 7, 2026
Merged

fix: allow deleting first time slot in availability schedule (#26448)#27569
sahitya-chandra merged 3 commits intocalcom:mainfrom
abhayymishraa:fix/26448-allow-delete-first-time-slot

Conversation

@abhayymishraa
Copy link
Copy Markdown
Contributor

@abhayymishraa abhayymishraa commented Feb 3, 2026

What does this PR do?

This PR fixes a bug where users could not delete the first time slot in their availability schedule or date overrides. Previously, only slots 2 and onwards showed a trash icon; the first slot only had the "+" button.

Changes:

  • Show the trash (delete) button on all time slots when there are 2 or more slots, including the first one.

  • When there is only 1 slot, no trash button is shown (you must keep at least one slot).

  • Fixes Calendar override remove first time #26448

  • Fixes CAL-XXXX (Linear issue number - should be visible at the bottom of the GitHub issue description)

Visual Demo (For contributors especially)

A visual demonstration is strongly recommended, for both the original and new change (video / image - any one).

Video Demo (if applicable):

Screen.Recording.2026-02-03.at.5.48.34.PM.mov

Image Demo (if applicable):

  • Add side-by-side screenshots of the original and updated change.
  • Highlight any significant change(s).

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

  1. Go to Settings → Availability (or /availability).
  2. Open any schedule.
  3. For any day, add 2 or 3 time slots using the "+" button.
  4. Check: All slots (including the first) should show a trash icon.
  5. Click trash on the first slot – it should be removed.
  6. When only 1 slot is left, the trash icon should disappear (you cannot delete the last slot).
  • Are there environment variables that should be set?
  • What are the minimal test data to have?
  • What is expected (happy path) to have (input and output)?
  • Any other important info that could help to test that PR

Checklist


Open with Devin

@github-actions github-actions bot added Low priority Created by Linear-GitHub Sync 🧹 Improvements Improvements to existing features. Mostly UX/UI labels Feb 3, 2026
@graphite-app graphite-app bot added the community Created by Linear-GitHub Sync label Feb 3, 2026
Copy link
Copy Markdown
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 1 file

Copy link
Copy Markdown
Member

@sahitya-chandra sahitya-chandra left a comment

Choose a reason for hiding this comment

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

LGTM

@sahitya-chandra sahitya-chandra added the run-ci Approve CI to run for external contributors label Feb 3, 2026
Copy link
Copy Markdown
Member

@sahitya-chandra sahitya-chandra left a comment

Choose a reason for hiding this comment

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

@abhayymishraa Can you move the trash icon to the right?

@abhayymishraa
Copy link
Copy Markdown
Contributor Author

image

@sahitya-chandra

@sahitya-chandra sahitya-chandra added run-ci Approve CI to run for external contributors and removed run-ci Approve CI to run for external contributors labels Feb 7, 2026
@sahitya-chandra sahitya-chandra removed the run-ci Approve CI to run for external contributors label Feb 7, 2026
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 4 additional findings.

Open in Devin Review

@sahitya-chandra sahitya-chandra added the run-ci Approve CI to run for external contributors label Feb 7, 2026
@sahitya-chandra sahitya-chandra enabled auto-merge (squash) February 7, 2026 13:11
@sahitya-chandra sahitya-chandra merged commit 0beadfa into calcom:main Feb 7, 2026
72 of 81 checks passed
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 🧹 Improvements Improvements to existing features. Mostly UX/UI Low priority Created by Linear-GitHub Sync ready-for-e2e run-ci Approve CI to run for external contributors size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Calendar override remove first time

2 participants