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.

Sign up for GitHub

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

Jump to bottom

Admin should be able to disable or enable users #769

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from
Open

Admin should be able to disable or enable users #769

wants to merge 8 commits into from

Conversation

fluturecode
Copy link
Contributor

@fluturecode fluturecode commented May 26, 2023 •
edited
Loading

Please run this along the backend PR at 'Shift3/dj-starter#67'

Changes

  1. Updated user model
  2. Updated user.api
  3. Updated userListView
  4. Updated userDetailForm

Purpose

To allow Admins to disable/enable all other users.

Testing

  1. Pull in the changes to your local copy of this branch and run yarn start
  2. Run the backend on branch admin-disable-users
  3. Check that you can disable user.
  4. Check that the user field is updated after disabling.
  5. Check to enable that user.
  6. Check that you can disable them again.

Screenshots

Screenshot 2023-05-26 at 11 58 25 AM

@fluturecode fluturecode changed the base branch from main to develop May 26, 2023 12:31
@fluturecode fluturecode mentioned this pull request May 26, 2023
Open
@fluturecode fluturecode marked this pull request as ready for review May 26, 2023 16:03
@fluturecode fluturecode requested a review from a team as a code owner May 26, 2023 16:03
@fluturecode fluturecode requested review from a team, michaelachrisco, krferrell and BennettSRobinson and removed request for a team May 26, 2023 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@michaelachrisco michaelachrisco michaelachrisco approved these changes

@krferrell krferrell Awaiting requested review from krferrell krferrell was automatically assigned from Shift3/pr-team

@BennettSRobinson BennettSRobinson Awaiting requested review from BennettSRobinson BennettSRobinson was automatically assigned from Shift3/pr-team

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fluturecode @michaelachrisco