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

allow multiple auth scopes #40

Merged
merged 1 commit into from
Jan 26, 2016
Merged

allow multiple auth scopes #40

merged 1 commit into from
Jan 26, 2016

Conversation

lawremi
Copy link
Contributor

@lawremi lawremi commented Jan 26, 2016

Sometimes we need to specify multiple auth scopes, like "read_only" and "compose". The gmail docs made it sound like compose implies read_only, but that does not appear to be true, in practice.

@jimhester
Copy link
Member

Using match.arg(several.ok = TRUE) does run us afoul of https://bugs.r-project.org/bugzilla3/show_bug.cgi?id=16659 (typos are silently ignored as long as one valid input is found). Regardless this is a good change and you are correct about the scoping, thanks!

jimhester added a commit that referenced this pull request Jan 26, 2016
allow multiple auth scopes
@jimhester jimhester merged commit 4e88d64 into r-lib:master Jan 26, 2016
@lawremi lawremi deleted the scope_fix branch January 29, 2016 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants