Commit e2f4a758 authored by Michael Kozono's avatar Michael Kozono

Merge branch 'nicolasdular/remove-signup-experiment' into 'master'

Remove signup flow experiment feature flag

See merge request gitlab-org/gitlab!47168
parents fd8caac1 b6c1128e
...@@ -51,7 +51,6 @@ module Registrations ...@@ -51,7 +51,6 @@ module Registrations
def requires_confirmation?(user) def requires_confirmation?(user)
return false if user.confirmed? return false if user.confirmed?
return false if Feature.enabled?(:soft_email_confirmation) return false if Feature.enabled?(:soft_email_confirmation)
return false if experiment_enabled?(:signup_flow)
true true
end end
......
...@@ -33,10 +33,6 @@ require 'zlib' ...@@ -33,10 +33,6 @@ require 'zlib'
module Gitlab module Gitlab
module Experimentation module Experimentation
EXPERIMENTS = { EXPERIMENTS = {
signup_flow: {
tracking_category: 'Growth::Acquisition::Experiment::SignUpFlow',
use_backwards_compatible_subject_index: true
},
onboarding_issues: { onboarding_issues: {
tracking_category: 'Growth::Conversion::Experiment::OnboardingIssues', tracking_category: 'Growth::Conversion::Experiment::OnboardingIssues',
use_backwards_compatible_subject_index: true use_backwards_compatible_subject_index: true
......
...@@ -7,7 +7,6 @@ require 'spec_helper' ...@@ -7,7 +7,6 @@ require 'spec_helper'
RSpec.describe Gitlab::Experimentation::EXPERIMENTS do RSpec.describe Gitlab::Experimentation::EXPERIMENTS do
it 'temporarily ensures we know what experiments exist for backwards compatibility' do it 'temporarily ensures we know what experiments exist for backwards compatibility' do
expected_experiment_keys = [ expected_experiment_keys = [
:signup_flow,
:onboarding_issues, :onboarding_issues,
:ci_notification_dot, :ci_notification_dot,
:upgrade_link_in_user_menu_a, :upgrade_link_in_user_menu_a,
......
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