Skip to content
This repository was archived by the owner on Dec 13, 2018. It is now read-only.

Rationalize SignInScheme in options #174

Closed
HaoK opened this issue Mar 5, 2015 · 3 comments
Closed

Rationalize SignInScheme in options #174

HaoK opened this issue Mar 5, 2015 · 3 comments
Assignees
@HaoK
Labels
3 - Done enhancement
Milestone
1.0.0-beta7

Comments

@HaoK
Copy link
Member

HaoK commented Mar 5, 2015

Oauth2/OIDC/ExternalAuthenticationOptions all define this, should see if we can consolidate these at all.
@HaoK HaoK self-assigned this Mar 5, 2015
@HaoK HaoK mentioned this issue Mar 5, 2015
Merged
@HaoK
Copy link
Member Author

HaoK commented Apr 23, 2015

Current thinking is ExternalAuthenticationOptions => SharedAuthenticationOptions

@Eilon Eilon added this to the 1.0.0-beta6 milestone Jun 11, 2015
@HaoK
Copy link
Member Author

HaoK commented Jul 15, 2015

Moving to beta 7 will do as part of API review PR

@HaoK HaoK modified the milestones: 1.0.0-beta7, 1.0.0-beta6 Jul 15, 2015
@HaoK
Copy link
Member Author

HaoK commented Jul 16, 2015

Renamed 5a2499e

@Eilon Eilon closed this as completed Aug 6, 2015
@Eilon Eilon added the 3 - Done label Aug 6, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@HaoK HaoK

Labels
3 - Done enhancement
Projects
None yet
Milestone
1.0.0-beta7
Development

No branches or pull requests
2 participants
@Eilon @HaoK