feat: Zoho Calendar #10429
feat: Zoho Calendar #10429
Conversation
Co-authored-by: Hariom Balhara <[email protected]>
@aar2dee2 can't reproduce |
| .filter((freebusy: FreeBusy) => freebusy.fbtype === "busy") | ||
| .map((freebusy: FreeBusy) => ({ | ||
| // using dayjs utc plugin because by default, dayjs parses and displays in local time, which causes a mismatch | ||
| start: dayjs.utc(freebusy.startTime, "YYYYMMDD[T]HHmmss[Z]").toISOString(), |
There was a problem hiding this comment.
This throws error when I enable 'Show me busy during non-work hours option', reason being the date format sent by Zoho changes in that case.
There was a problem hiding this comment.
Also, I think it makes sense to document somewhere that one needs to enable this free busy option for events to be considered as busy.
| }), | ||
| }); | ||
|
|
||
| const eventResponse = await this.fetcher(`/calendars/${calendarId}/events/${uid}?${query}`, { |
There was a problem hiding this comment.
If for the booking being rescheduled in Cal.com, Zoho calendar event doesn't exist due to some reason(I deleted that in my case), there comes an error in rescheduling the event on Zoho.
I think we should handle it gracefully, by creating the event on rescheduled time.
Though not a blocker, we can create a ticket for it and work on it later.
hariombalhara
removed
❗️ migrations
|
Thankyou @murtajaziad 🙏 Overall it works well, left a few comments but they can be addressed in a followup. @aar2dee2 if you don't have any more concerns let's merge it. |
yes, let's proceed |
fixed when |
Co-authored-by: Hariom Balhara <[email protected]> Co-authored-by: alannnc <[email protected]> Co-authored-by: Joe Au-Yeung <[email protected]> Co-authored-by: Joe Au-Yeung <[email protected]> Co-authored-by: Peer Richelsen <[email protected]> Co-authored-by: aar2dee2 <[email protected]>
What does this PR do?
An app to integrate Zoho Calendar and Cal.com using the official Zoho Calendar API.
This PR is a reopen of already-existing closed PR #8144 with all the required changes done.
Fixes #5451
Environment: Staging(main branch)
Type of change
How should this be tested?
No automated tests are set up for this.
You can manually test the integration by following these steps:
Checklist
/claim #5451