Closed dericksozo closed 5 months ago
Thanks for the feedback, @PazYanoverr. @mulygottlieb, I'll keep posted for yours!
@mulygottlieb - Can you please review before publishing?
@PazYanoverr @dericksozo I added my comment but all in all it looks great.
One more thing that we need to think about (not necessarily a blocker for this PR) - in .NET this works a bit differently and we need to document that separately (the names of the Auth plugin(s) are different, and the process behind the scenes of using ASP.NET Core Identity is a bit different). We can decide to open a ticket about that for later on so we don't block this PR any longer.
Thanks @mulygottlieb ! @dericksozo - Can you please apply Muly's comments and open a separate ticket for the .NET part?
Yep, I'll get on it, @PazYanoverr. Thanks @mulygottlieb;
(docs): Implementing the necessary changes for the User Entity page based on Paz's feedback.
This PR takes care of Issue #482.