-
Notifications
You must be signed in to change notification settings - Fork 25.5k
[Zoom] Update existing scopes with granular scopes #113994
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
Conversation
Documentation preview: |
Pinging @elastic/es-docs (Team:Docs) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanx! Just one minor syntax fix required 🚀
@moxarth-elastic few minor process points for future PRs in the Elasticsearch repo:
|
Pinging @elastic/ingestion-team (Team:Search - Extract & Transform) |
Pinging @elastic/search-eng (Team:SearchOrg) |
Co-authored-by: Liam Thompson <[email protected]>
Noted, Thanks! |
@leemthompo can you please re-run the CI and merge the PR? I don't have merge access. |
@elasticmachine update branch |
@moxarth-elastic how many Elastic versions should this be backported to? |
💔 Backport failedThe backport operation could not be completed due to the following error:
You can use sqren/backport to manually backport by running |
💚 All backports created successfully
Questions ?Please refer to the Backport tool documentation |
(cherry picked from commit e1bba9b)
(cherry picked from commit e1bba9b) Co-authored-by: moxarth-elastic <[email protected]>
we can do it for 8.11.0 and later versions - elastic/connectors#1504. |
Part of elastic/connectors#2818
Existing permissions are deprecated in the Zoom developer app, replacing them with the new granular scopes - https://developers.zoom.us/docs/integrations/oauth-scopes-overview/.
Fyi, the
cloud_recording:read:list_user_recordings:admin
scope is not tested due to not having a pro account.gradle check
?