A todo list for new ASP.NET Core projects

Re:A todo list for new ASP.NET Core projects by Roonerelli on Thursday February 08, 2018 @04:17AM (#16331070)

the generic parameter in this class is never used

public class ApplicationUserManager<TUser>

Plus you could just have IHttpContextAccessor injected instead and have extension methods off the ClaimsPrincipal to retrieve specific claims

but ... horses for courses I guess

[ Parent | on HN ]

Re:A todo list for new ASP.NET Core projects by biarity on Thursday February 08, 2018 @04:37AM (#16331135)

I think your method is much cleaner actually. Just updated :)!

[ Parent | on HN ]

Re:A todo list for new ASP.NET Core projects by Lord_Zero on Thursday February 08, 2018 @12:12PM (#16333671)

Thanks for reading these comments and updating your code! I personally appreciate the effort.

[ Parent | on HN ]