Skip to content
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

Potential error in Wiki for Setup Authentication #78

Closed
roos-robert opened this issue Mar 8, 2023 · 1 comment
Closed

Potential error in Wiki for Setup Authentication #78

roos-robert opened this issue Mar 8, 2023 · 1 comment

Comments

@roos-robert
Copy link

Hi Jon! Thank you for this amazing project and the hard work you are putting into it!
I was working on replacing IdentityUser with ApplicationUser in a app using AuthP, checked the wiki for what needed to be changed in my code, over here: https://github.com/JonPSmith/AuthPermissions.AspNetCore/wiki/Setup-Authentication

On point 3 it says the following:

There are some other code used in the examples that only work with UserManager (see below). If you want to use these you need to copy these too:

HostedServiceAddAspNetUsers
AspNetUserExtension, specifically ListAllAspNetUsers and CheckAddNewUserAsync

However it seems like ListAllAspNetUsers and HostedServiceAddAspNetUsers are no longer in the project (maybe renamed?). Just thought I'd point it out since this is a feature I'm guessing a lot of people will be/are using to extend the IdentityUser.

Have a great rest of your week! :)

@JonPSmith
Copy link
Owner

Hi @rr222cy,

Thanks for pointing out this out-of-date documentarian, which I have fixed. When I define the task list for a new version I have to add documentation tasks too, otherwise I won't remember. This one I missed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants