Skip to content
This repository has been archived by the owner on Jan 26, 2025. It is now read-only.

Remove array check on scopes #120

Merged
merged 1 commit into from
Feb 14, 2018
Merged

Conversation

jmelberg-okta
Copy link
Contributor

BREAKING CHANGE

  • Requires scopes passed in as a custom parameter to be a single string.
    • openid profile email custom
    • ['openid', 'profile', 'email', 'custom']

@jmelberg-okta jmelberg-okta added this to the 1.0.0 - Angular & Vue milestone Feb 7, 2018
@cmckni3
Copy link

cmckni3 commented Feb 7, 2018

@jmelberg-okta Are these pull requests going to be merged soon?

@robertjd
Copy link
Contributor

robertjd commented Feb 7, 2018

Hi @cmckni3 , thank you for your active interest, your feedback is very useful for us as we iterate 👍 . We're collecting feedback on the current 0.x version of this library and starting to work towards a 1.0 release that will introduce any improvements as breaking changes. I can't guarantee a timeline but we are actively working on it this month. Please continue to watch the 1.0.0-angular branch, as that is where all changes will be merged. Once we have collected all desired changes for this major rev we will merge to master. If you have a fork that solves your needs I would continue using that for the time being.

@cmckni3
Copy link

cmckni3 commented Feb 7, 2018

Should I change the branch of my PR #116 @robertjd?

@jmelberg-okta jmelberg-okta merged commit fbfd759 into 1.0.0-angular Feb 14, 2018
@jmelberg-okta jmelberg-okta deleted the jm-remove-scope-array branch February 14, 2018 22:12
@robertjd
Copy link
Contributor

@cmckni3 RE your question about #116, I haven't had a chance to look at that in detail. If there are no breaking changes then you can keep your PR as going to master. Thanks!

jmelberg-okta added a commit that referenced this pull request Mar 6, 2018
jmelberg-okta added a commit that referenced this pull request Mar 20, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants