Skip to content
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

Removed calendar scope, we shouldn't need it. #236

Merged
merged 1 commit into from
Jun 3, 2021
Merged

Removed calendar scope, we shouldn't need it. #236

merged 1 commit into from
Jun 3, 2021

Conversation

emrysal
Copy link
Contributor

@emrysal emrysal commented Jun 3, 2021

@jfernandogt Regarding the Google Verification process, and looking at the documentation and through tests I concluded https://www.googleapis.com/auth/calendar.readonly covers the GET calendarlist action. As you explicitly added this scope I'm tagging you in this PR.

The access gained with https://www.googleapis.com/auth/calendar is too broad, so the verification process would defacto fail if we were to keep this scope in the repository.

@vercel
Copy link

vercel bot commented Jun 3, 2021

@emrysal is attempting to deploy a commit to the calendso Team on Vercel.

A member of the Team first needs to authorize it.

@baileypumfleet baileypumfleet merged commit 8794c9f into calcom:main Jun 3, 2021
KATT pushed a commit that referenced this pull request Sep 3, 2021
Bumps [core-js](https://github.com/zloirock/core-js/tree/HEAD/packages/core-js) from 3.9.1 to 3.10.0.
- [Release notes](https://github.com/zloirock/core-js/releases)
- [Changelog](https://github.com/zloirock/core-js/blob/master/CHANGELOG.md)
- [Commits](https://github.com/zloirock/core-js/commits/v3.10.0/packages/core-js)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
zomars pushed a commit that referenced this pull request May 23, 2023
Hotfix attempt to fix fix invalid availability _post schema in swagger
@PeerRich PeerRich added the core area: core, team members only label Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core area: core, team members only
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants