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

Update supported scopes #712

Merged
merged 1 commit into from
Oct 14, 2021
Merged

Conversation

boekkooi-lengoo
Copy link
Contributor

This aligns the scopes with the supported list provided via https://developer.okta.com/docs/guides/implement-oauth-for-okta/scopes/#scope-naming.

@bogdanprodan-okta bogdanprodan-okta merged commit faf33c8 into okta:master Oct 14, 2021
@boekkooi-lengoo boekkooi-lengoo deleted the patch-1 branch October 15, 2021 07:03
@@ -10,10 +10,13 @@ import (
"github.com/okta/okta-sdk-golang/v2/okta"
)

# validScopes is a list of supported scopes as per https://developer.okta.com/docs/guides/implement-oauth-for-okta/scopes/.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

# typo /cc @bogdanprodan-okta

Copy link
Contributor Author

@boekkooi-lengoo boekkooi-lengoo Oct 17, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤦 Sorry about that I was writing to much YAML, a fix is available under #718 or as part of #716

boekkooi-lengoo added a commit to boekkooi-lengoo/terraform-provider-okta that referenced this pull request Oct 17, 2021
Fixes the error `invalid character U+0023 '#'` as spotted by @monde 

This issue was introduced in okta#712 and sadly not caught by a linter in the CI.
bogdanprodan-okta pushed a commit that referenced this pull request Oct 18, 2021
Fixes the error `invalid character U+0023 '#'` as spotted by @monde 

This issue was introduced in #712 and sadly not caught by a linter in the CI.
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.

3 participants