Skip to content

User admin approval - Approve users pending approval via admin UI

What does this MR do?

For #257886 (closed)

This is part of an epic and a good bunch of work on this feature has already been merged - &4491

**A short walkthru video explaining the change is available here ** (Update: This is slightly outdated now, as the approval flow does not include force confirming the user anymore)

This change allows the following operations via the admin UI.

  • Shows the users pending approval under "Pending approval" tab.
  • Allows to approve a pending user. This step
    • activates the user
    • send confirmation instruction to the users email if they are not confirmed yet
    • accepts their pending invitations, if the user satisfies all prerequisites for a successful login and has their email confirmed.
  • Allows to block a pending user.
  • Allows to delete a pending user.

Everything is behind a feature flag, which will be removed soon with #258980 (closed), hence there is no changelog.

I will ask for review of UI text from Technical Writing & add documentation for the feature in a different MR (as this is behind a feature flag now, these will not be shown anyway)

Screenshots

New "Pending approval" tab shows users pending approval:

Screenshot_2020-10-12_at_11.04.01_AM

Operations available on pending users - they can be approved, blocked or deleted:

Screenshot_2020-10-14_at_11.30.19_AM

Show pending user page - has the same operations available: (this is for a user that does not have their email confirmed yet)

Screenshot_2020-10-14_at_11.31.13_AM

Show pending user page - has the same operations available: (this is for a user that hash their email confirmed already)

Screenshot_2020-10-14_at_11.30.58_AM

After approving a user:

Screenshot_2020-10-14_at_11.31.27_AM

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Manoj M J

Merge request reports