fix(calendar): allow editing and clearing appointment recurrence #248

Merged
vas2000-work merged 1 commit from fix/MAPPS-236-edit-recurrence-rule into main 2026-06-17 23:01:42 +02:00
Owner

The recurrence RRULE was create-only: the input was gated out of edit mode
with if !is_edit, the form signal always seeded empty, and
UpdateAppointmentRequest had no recurrence_rule field, so an existing rule
could neither be changed nor cleared.

Seed the recurrence signal from the appointment, always show the input, add
recurrence_rule to UpdateAppointmentRequest, and send it on update. The
update field is always serialized (sent as null when empty) so emptying the
input clears the rule rather than leaving the prior value in place.

#MAPPS-236

The recurrence RRULE was create-only: the input was gated out of edit mode with `if !is_edit`, the form signal always seeded empty, and UpdateAppointmentRequest had no recurrence_rule field, so an existing rule could neither be changed nor cleared. Seed the recurrence signal from the appointment, always show the input, add recurrence_rule to UpdateAppointmentRequest, and send it on update. The update field is always serialized (sent as null when empty) so emptying the input clears the rule rather than leaving the prior value in place. #MAPPS-236
fix(calendar): allow editing and clearing appointment recurrence
All checks were successful
Check / clippy + fmt + tests (pull_request) Successful in 1m41s
Create release / Create release from merged PR (pull_request) Has been skipped
72532259f7
The recurrence RRULE was create-only: the input was gated out of edit mode
with `if !is_edit`, the form signal always seeded empty, and
UpdateAppointmentRequest had no recurrence_rule field, so an existing rule
could neither be changed nor cleared.

Seed the recurrence signal from the appointment, always show the input, add
recurrence_rule to UpdateAppointmentRequest, and send it on update. The
update field is always serialized (sent as null when empty) so emptying the
input clears the rule rather than leaving the prior value in place.

#MAPPS-236
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
psa-systems/mokosh-apps!248
No description provided.