-
Notifications
You must be signed in to change notification settings - Fork 20
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
Comments
@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. |
@tyler-dane Can you clarify this one please? |
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 |
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.
The text was updated successfully, but these errors were encountered: