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

IdentityBuilder does not contain AddSignInManager however it do contain AddUserManager #565

Closed
ianido opened this issue Sep 16, 2015 · 2 comments

Comments

@ianido
Copy link

ianido commented Sep 16, 2015

If you want to implement your own SignInManager and UserManager you have to activate them like:

services.AddScoped(typeof(MyUserManager<IUser>));
services.AddScoped(typeof(MySignInManager<IUser>));

services.AddIdentity<ApplicationUser, ApplicationRole>()

so what is the purpose of: AddUserManager

services.AddIdentity<ApplicationUser, ApplicationRole>()
.AddUserManager<MyUserManager<IUser>>() 

it does not work.

would be better if we can declare them like:

services.AddIdentity<ApplicationUser, ApplicationRole>()
.AddSignInManager<MySignInManager<IUser>>() 
.AddUserManager<MyUserManager<IUser>>()

or simply remove the AddUserManager method.

@rustd rustd added this to the Backlog milestone Sep 21, 2015
@rustd
Copy link

rustd commented Sep 21, 2015

We are not going to have this for RTM but will consider it for the future.

@HaoK HaoK modified the milestones: 1.0.1, Backlog Jul 1, 2016
@HaoK HaoK self-assigned this Jul 1, 2016
This was referenced Jul 1, 2016
@HaoK HaoK added 3 - Done and removed 2 - Working labels Aug 19, 2016
@HaoK
Copy link
Member

HaoK commented Aug 19, 2016

27f021a

@HaoK HaoK closed this as completed Aug 19, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants