Commit 7bf9aef1 authored by Peter Leitzen's avatar Peter Leitzen

Merge branch 'remove-unused-url-param' into 'master'

Remove unused URL param

See merge request gitlab-org/gitlab!31343
parents 5b639a29 3fd7c847
......@@ -24,7 +24,7 @@ class SubscriptionsController < ApplicationController
def new
return if current_user
store_location_for_user
store_location_for :user, request.fullpath
redirect_to new_user_registration_path(redirect_from: 'checkout')
end
......@@ -85,15 +85,6 @@ class SubscriptionsController < ApplicationController
Gitlab::SubscriptionPortal::Client
end
def store_location_for_user
redirect_url = url_for(safe_params.merge(experiment_started: true))
store_location_for :user, redirect_url
end
def experiment_already_started?
params[:experiment_started].present?
end
def customer_portal_new_subscription_url
"#{EE::SUBSCRIPTIONS_URL}/subscriptions/new?plan_id=#{params[:plan_id]}&transaction=create_subscription"
end
......
......@@ -15,7 +15,7 @@ describe SubscriptionsController do
it 'stores subscription URL for later' do
subject
expected_subscription_path = new_subscriptions_path(experiment_started: true, plan_id: 'bronze_id')
expected_subscription_path = new_subscriptions_path(plan_id: 'bronze_id')
expect(controller.stored_location_for(:user)).to eq(expected_subscription_path)
end
......
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