-
Notifications
You must be signed in to change notification settings - Fork 50
Description
@tyler-dane It looks like this user has google calendars in their calendar list with the following fields undefined.
- etag
- accessRole
- conferenceProperties
- defaultReminders
We can relax the schema for these fields excepting the "accessRole" field.
Note
According to https://developers.google.com/workspace/calendar/api/v3/reference/calendarList#resource, these fields are not optional, so they should be included as part of the payload for each calendarlist item saved in Compass or Comming from Google calendar.
As part of my findings, this method here
canDoIncrementalSyncneeds to be rethought. It is not a good method to determine whether a user should go through the signup or login phase, and allows for potential duplicate user accounts becuase we do not currently have mechanisms in place currently to prevent duplicate account creation at the database level. There is a schema change story to mitigate this #1038, but we still need to move away from determining the auth path from the ability of a user to do incremental sync.
Originally posted by @victor-enogwe in #1145
Metadata
Metadata
Assignees
Labels
Type
Projects
Status