youssef1313 / askq Goto Github PK

• Add Name field for the user
• Make UserName different than Email
• Allow for profile photo.

https://docs.microsoft.com/en-us/aspnet/core/security/authentication/add-user-data?view=aspnetcore-3.0&tabs=visual-studio

Currently, the Services exist in the web project and depend on AppDbContext. However, the Services in eShopOnWeb are placed in ApplicationCore and doesn't depend on AppDbContext.

For complex applications, I think the Services may contain business logic, which wouldn't be ideal to be placed in the web layer.

cc: @fiseni If you got time, I hope you can provide help in this area

You might start refactoring the usage of IdentityUser in the models:

• Models should not contain properties of type IdentityUser, just id of the user (in this case guid). Make the property type string. The responsibility of the services would be to reconcile the id with the appropriate user persisted in db (if required for some operation)
• ViewModels should not contain IdentityUser properties either. Here just add properties that you would show on the view, name, email or whatever. Dont expose more information than needed.

Ps. I'll add other hints tomorrow, kinda late here :)

https://docs.microsoft.com/en-us/aspnet/core/security/authentication/accconfirm?view=aspnetcore-3.1&tabs=visual-studio

Track the warnings that were silenced in #23 (6fc4874, 17ba109)

Extend the IdentityUser to have an option whether to allow/disallow anonymous questions.