fix: disable booking actions for cancelled/rejected/past bookings #26926
+13
−6
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Fixes inconsistent disabled state for booking actions. Several actions (
Edit location,Add Guests,Reroute,Reassign) were hardcoded todisabled: false, allowing users to see enabled buttons on cancelled, rejected, or past bookings.Users could see enabled "Edit Location" or "Add Guests" buttons on bookings that were already cancelled/rejected/past, which doesn't make sense functionally.
Changes -
change_location,add_members,reroute,reassignnow useisActionDisabled()instead of hardcodeddisabled: falsecancelaction now properly checks forisCancelledandisRejectedstatesreschedule_requestnow checks forisCancelledandisRejectedstatesVisual Demo (For contributors especially)
Video Demo (if applicable):
cursorful-video-1768558098829.mp4
cursorful-video-1768557741853.mp4
Mandatory Tasks (DO NOT REMOVE)
How should this be tested?
Tested locally and verified with before/after screen recordings.