Commit df875027 authored by manojmj's avatar manojmj

Enable feature flag `admin_approval_for_new_user_signups ` by default

This change enables the feature flag
`admin_approval_for_new_user_signups `
by default
parent 3d77669a
...@@ -300,7 +300,7 @@ class Admin::UsersController < Admin::ApplicationController ...@@ -300,7 +300,7 @@ class Admin::UsersController < Admin::ApplicationController
end end
def check_admin_approval_feature_available! def check_admin_approval_feature_available!
access_denied! unless Feature.enabled?(:admin_approval_for_new_user_signups) access_denied! unless Feature.enabled?(:admin_approval_for_new_user_signups, default_enabled: true)
end end
end end
......
...@@ -216,7 +216,7 @@ class RegistrationsController < Devise::RegistrationsController ...@@ -216,7 +216,7 @@ class RegistrationsController < Devise::RegistrationsController
end end
def set_user_state def set_user_state
return unless Feature.enabled?(:admin_approval_for_new_user_signups) return unless Feature.enabled?(:admin_approval_for_new_user_signups, default_enabled: true)
return unless Gitlab::CurrentSettings.require_admin_approval_after_user_signup return unless Gitlab::CurrentSettings.require_admin_approval_after_user_signup
resource.state = BLOCKED_PENDING_APPROVAL_STATE resource.state = BLOCKED_PENDING_APPROVAL_STATE
......
...@@ -9,7 +9,7 @@ ...@@ -9,7 +9,7 @@
Sign-up enabled Sign-up enabled
.form-text.text-muted .form-text.text-muted
= _("When enabled, any user visiting %{host} will be able to create an account.") % { host: "#{new_user_session_url(host: Gitlab.config.gitlab.host)}" } = _("When enabled, any user visiting %{host} will be able to create an account.") % { host: "#{new_user_session_url(host: Gitlab.config.gitlab.host)}" }
- if Feature.enabled?(:admin_approval_for_new_user_signups) - if Feature.enabled?(:admin_approval_for_new_user_signups, default_enabled: true)
.form-group .form-group
.form-check .form-check
= f.check_box :require_admin_approval_after_user_signup, class: 'form-check-input' = f.check_box :require_admin_approval_after_user_signup, class: 'form-check-input'
......
...@@ -30,7 +30,7 @@ ...@@ -30,7 +30,7 @@
= link_to admin_users_path(filter: "blocked") do = link_to admin_users_path(filter: "blocked") do
= s_('AdminUsers|Blocked') = s_('AdminUsers|Blocked')
%small.badge.badge-pill= limited_counter_with_delimiter(User.blocked) %small.badge.badge-pill= limited_counter_with_delimiter(User.blocked)
- if Feature.enabled?(:admin_approval_for_new_user_signups) - if Feature.enabled?(:admin_approval_for_new_user_signups, default_enabled: true)
= nav_link(html_options: { class: "#{active_when(params[:filter] == 'blocked_pending_approval')} filter-blocked-pending-approval" }) do = nav_link(html_options: { class: "#{active_when(params[:filter] == 'blocked_pending_approval')} filter-blocked-pending-approval" }) do
= link_to admin_users_path(filter: "blocked_pending_approval") do = link_to admin_users_path(filter: "blocked_pending_approval") do
= s_('AdminUsers|Pending approval') = s_('AdminUsers|Pending approval')
......
---
title: Introduce 'admin approvals for new user signups' feature
merge_request: 45233
author:
type: added
...@@ -4,4 +4,4 @@ introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/43827 ...@@ -4,4 +4,4 @@ introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/43827
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/258980 rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/258980
type: development type: development
group: group::access group: group::access
default_enabled: false default_enabled: true
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment