You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Instead of actually deleting calendars we should keep them, and just mark them as deleted, like with flag isDeleted: true. (We need to keep calendars because maybe we have some interview scheduled with calendar we are deleting, so better we have access token to that calendar in case we need to make any change for already scheduled meeting).
When we get UserMeetingSettings using endpoint GET /taas/user-meeting-settings/:userId we should NOT return calendars marked as deleted.
When we connect a new calendar and there is existent deleted calendar with the same id we have to update its details and mark it as not deleted isDeleted: false. So there should be never duplicate calendars with the same id even if it was deleted. When we connect already deleted calendar this might mean that we re-connect previously deleted calendar. I'm not sure if this situation would really happen or Nylas would alway give a new id but better support it.
maxceem
changed the title
Don't delete connected calendars, just flag as deleted
[$50] Don't delete connected calendars, just flag as deleted
Oct 23, 2021
@maxceem, when we delete a calendar in the previous flow, we didn't used to delete it from Nylas server, but we deleted from the UserMeetingSettings.nylasCalendars record. So Nylas won't give a new id.
Branch
feature/interview-update
.We have endpoint to delete connected calendars which was implemented in this challenge https://www.topcoder.com/challenges/af9a1041-7a78-484d-87d3-5aec9e87b578?tab=details
DELETE /taas/user-meeting-settings/:userId/calendars/:calendarId
isDeleted: true
. (We need to keep calendars because maybe we have some interview scheduled with calendar we are deleting, so better we have access token to that calendar in case we need to make any change for already scheduled meeting).GET /taas/user-meeting-settings/:userId
we should NOT return calendars marked as deleted.id
we have to update its details and mark it as not deletedisDeleted: false
. So there should be never duplicate calendars with the sameid
even if it was deleted. When we connect already deleted calendar this might mean that we re-connect previously deleted calendar. I'm not sure if this situation would really happen or Nylas would alway give a newid
but better support it.The text was updated successfully, but these errors were encountered: