Skip to content
This repository has been archived by the owner on Jul 18, 2023. It is now read-only.

Edits to User Management Page #198

Closed
kjacobsen16 opened this issue Aug 7, 2019 · 0 comments
Closed

Edits to User Management Page #198

kjacobsen16 opened this issue Aug 7, 2019 · 0 comments
Assignees

Comments

@kjacobsen16
Copy link
Contributor

kjacobsen16 commented Aug 7, 2019

I culled the response field list, it will now be:
fields = ('id', 'username', 'first_name', 'last_name', 'email', 'is_staff', 'is_active',)
the other fields are not used by the system currently and would only confuse the end user, and the is_superuser field is not relevant to non-superusers
oh, and if a user is active but not staff, they cannot create new users
so for that last situation, you may want to test the user's is_staff value, and if it's false hide the Create User button
since that user would only be able to edit their own values, and even then only a subset (specifically: 'username', 'first_name', 'last_name', 'email')
so I guess you'd also want to hide the other fields in the editor modal for that user
they should still be able to see all users and filter them
but let's still only let them edit their own record
might need to put an explicit note in the HTML on that page explicitly explaining that

@kjacobsen16 kjacobsen16 self-assigned this Aug 7, 2019
kjacobsen16 added a commit that referenced this issue Aug 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant