Skip to content

Turn recurring sidebar event into regular event after dropping on grid #297

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
tyler-dane opened this issue Mar 10, 2025 · 3 comments · Fixed by #349
Closed

Turn recurring sidebar event into regular event after dropping on grid #297

tyler-dane opened this issue Mar 10, 2025 · 3 comments · Fixed by #349
Assignees

Comments

@tyler-dane
Copy link
Contributor

tyler-dane commented Mar 10, 2025

Currently, the recurrence values of a sidebar event are maintained after dropping it on the grid.
For simplicity and clarity, handle this case as a regular event by removing the recurrence properties.

This will let the user understand that the event is now a regular event and its original recurrence from the sidebar no longer applies now that it's on the main grid.

@tyler-dane
Copy link
Contributor Author

@that-one-arab I assigned this to you because I've still got my hands full with the recurrence backend changes, and this can be done independent of that.

@that-one-arab
Copy link
Contributor

@tyler-dane Can you clarify this one please?

  • Are we to remove the recurrence field from the event object?
    Image
  • Or are we to remove the recurrence fields in the event form?
    Image

@tyler-dane
Copy link
Contributor Author

The former - we need to strip the irrelevant recurrence properties after dropping, so that they don't make it into the DB as part of the update request. This'll help us avoid future scenarios, where an event seems like a recurrence but actually isn't

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment