Skip to content

Add docs for: `Optional admin approval for local user sign up` feature

What does this MR do?

Add documentation for the feature built for &4491 - Optional admin approval for local user sign up

A short walkthru video explaining the change is available here (Update: Slightly outdated now, as the discussion @ !44877 (comment 428615050) lead us to removing the "force confirm email" part from the approval flow)

Screenshots

  1. Sign up restriction

Message: When enabled, any user visiting http://localhost:3000/users/sign_in and creating an account will have to be explicitly approved by an admin before they can sign in. This setting is effective only if sign-ups are enabled.

Screenshot_2020-10-13_at_2.38.56_PM

  1. A new user signs up:

Message - "You have signed up successfully. However, we could not sign you in because your account is awaiting approval from your GitLab administrator."

Screenshot_2020-10-14_at_12.46.22_PM

  1. This user pending approval tries to sign in:

Message - "Your account is pending approval from your GitLab administrator and hence blocked. Please contact your GitLab administrator if you think this is an error."

Screenshot_2020-10-13_at_2.36.06_PM

  1. This user pending approval tries to access API/Git:

Message - "Your account is pending approval from your administrator and hence blocked."

Screenshot_2020-10-13_at_5.11.39_PM

  1. Approval page for admin:

Message:

This user has requested access

Approved users can:

  • Log in
  • Access Git repositories
  • Access the API
  • Be added to groups and projects

Screenshot_2020-10-14_at_12.45.09_PM

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