Skip to content
This repository has been archived by the owner on Oct 11, 2019. It is now read-only.

fix change password doesn't work #289

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

AliveMen
Copy link
Contributor

add and use NonEditableUsers setting

@AliveMen AliveMen requested a review from tatarincev May 28, 2019 12:39
Copy link
Collaborator

@tatarincev tatarincev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you explain how these changes related to this particular issue?

@AliveMen
Copy link
Contributor Author

Admin didn't have ability to change password because

services.AddSecurityServices(options =>
            {
                options.NonEditableUsers = new[] { "admin" };
            });

I've removed options.NonEditableUsers = new[] { "admin" }; from startup and add NonEditableUsers setting like in platform v2.

@AliveMen AliveMen force-pushed the fix_not_allowed_to_change_password branch from 50b36a8 to 475d174 Compare May 29, 2019 06:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants