Skip to content

fix: disable canceling booking if no internal note is selected#18926

Merged
sean-brydon merged 3 commits intomainfrom
fix/make-internal-note-mandatory
Jan 28, 2025
Merged

fix: disable canceling booking if no internal note is selected#18926
sean-brydon merged 3 commits intomainfrom
fix/make-internal-note-mandatory

Conversation

@sean-brydon
Copy link
Copy Markdown
Member

What does this PR do?

This PR prevents the cancel button being clicked if no internal note is selected

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?

Create internal notes in team settings
create a booking
cancel booking
Try without a cancelation reason and without internal note

Checklist

  • I haven't read the contributing guide
  • My code doesn't follow the style guidelines of this project
  • I haven't commented my code, particularly in hard-to-understand areas
  • I haven't checked if my changes generate no new warnings

@graphite-app graphite-app Bot requested a review from a team January 27, 2025 17:39
@keithwillcode keithwillcode added consumer core area: core, team members only labels Jan 27, 2025
@dosubot dosubot Bot added bookings area: bookings, availability, timezones, double booking 🐛 bug Something isn't working labels Jan 27, 2025
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Jan 27, 2025

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (01/27/25)

1 reviewer was added to this PR based on Keith Williams's automation.

"Add ready-for-e2e label" took an action on this PR • (01/28/25)

1 label was added to this PR based on Keith Williams's automation.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 27, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Jan 28, 2025 8:02am
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Jan 28, 2025 8:02am

@github-actions
Copy link
Copy Markdown
Contributor

E2E results are ready!

@sean-brydon sean-brydon merged commit cc94fb4 into main Jan 28, 2025
MuhammadAimanSulaiman pushed a commit to hit-pay/cal.com that referenced this pull request Feb 25, 2025
@keithwillcode keithwillcode deleted the fix/make-internal-note-mandatory branch September 17, 2025 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bookings area: bookings, availability, timezones, double booking 🐛 bug Something isn't working consumer core area: core, team members only ready-for-e2e

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants