-
Notifications
You must be signed in to change notification settings - Fork 418
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 AuthorizedState enum #1062
Removed AuthorizedState enum #1062
Conversation
It seems the library uses "authorized" and "authenticated" nearly interchangeably. I initially authored this with On that note, is there any reason why "authorized" is used in place of "authenticated" in other places in the application? To my knowledge, these are different things. |
@JerrettDavis LGTM |
Looks good to me as well. I am sure @damienbod will answer your questions. Could you in the meantime
Thanks! |
…-auth-oidc-client into feature/remove-authorized-state-enum
I have corrected the target branch, merged the Thanks! |
AuthorizedState
should only ever be true or false, I have removed theAuthorizedState
enum per issue #755.