fix: update zoom integration doc to include user settings scope#16724
fix: update zoom integration doc to include user settings scope#16724anikdhabal merged 5 commits intocalcom:mainfrom
Conversation
|
@vijayraghav-io is attempting to deploy a commit to the cal Team on Vercel. A member of the Team first needs to authorize it. |
Graphite Automations"Add consumer team as reviewer" took an action on this PR • (09/19/24)1 reviewer was added to this PR based on Keith Williams's automation. "Add community label" took an action on this PR • (09/19/24)1 label was added to this PR based on Keith Williams's automation. "Add ready-for-e2e label" took an action on this PR • (09/21/24)1 label was added to this PR based on Keith Williams's automation. |
|
Just wanted to confirm, we don't have that working currently in app.cal.com, and to make it work we have to add scope to the currently used credentials right? |
Co-authored-by: Amit Sharma <74371312+Amit91848@users.noreply.github.com>
Scope needs to be updated in ZoomAppMarketplace for Cal App, nothing to update on Cal's credential. |
| 7. Now copy the Client ID and Client Secret to your `.env` file into the `ZOOM_CLIENT_ID` and `ZOOM_CLIENT_SECRET` fields. | ||
| 8. Set the "OAuth Redirect URL" under "OAuth Information" as `<Cal.com URL>/api/integrations/zoomvideo/callback` replacing Cal.com URL with the URI at which your application runs. | ||
| 9. Also add the redirect URL given above as an allow list URL and enable "Subdomain check". Make sure, it says "saved" below the form. | ||
| 10. You don't need to provide basic information about your app. Instead click on "Scopes" and then on "+ Add Scopes". On the left, click the category "Meeting" and check the scope `meeting:write:meeting` and `user:read:settings`. |
There was a problem hiding this comment.
user:read:settings is not part of the 'meeting' category, so what's the point of using 'and'? I suggested you make two points regarding this
There was a problem hiding this comment.
yeah that's right, that's why i had added it in a separate point.
There was a problem hiding this comment.
@anikdhabal , updated can you please check.
Amit91848
left a comment
Amit91848
left a comment
There was a problem hiding this comment.
LGTM! Thanks @vijayraghav-io
Thankyou! @Amit91848 🙏 |
3 participants
What does this PR do?
Refer: #15489
Video showing to add
user:read:settingsscope in zoomAppMarketplacehttps://www.loom.com/share/d1b8f6910ca946f7a729a80c78167da0?sid=7950171a-681c-45c4-9d7f-55aff7afe500
Video showing working as expected currently after #15489 merged
https://www.loom.com/share/48799d1d4af44de8ae82a6fff7353b0f?sid=8efbe50b-af43-4918-bbb7-4864a513b5ff
Mandatory Tasks (DO NOT REMOVE)