dev(auth-v2): rename variable to debtcollective_after_signup_redirect…

…_url
debtcollective · Feb 11, 2021 · d1e9ded · d1e9ded
1 parent b1b6ff4
commit d1e9ded
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 5 deletions.
diff --git a/config/locales/server.en.yml b/config/locales/server.en.yml
@@ -4,4 +4,4 @@ en:
     debtcollective_algolia_app_id: 'Algolia Places App ID'
     debtcollective_algolia_api_key: 'Algolia Places API key'
     debtcollective_algolia_api_key: 'Algolia Places API key'
-    debtcollective_redirect_url_after_signup: 'Redirect URL after user accepts an invite. ex https://debtcollective.org/hub'
+    debtcollective_after_signup_redirect_url: 'Redirect URL after user accepts an invite. ex https://debtcollective.org/hub'
diff --git a/config/settings.yml b/config/settings.yml
@@ -22,6 +22,6 @@ plugins:
   debtcollective_solidarity_message_title:
     default: "Solidarity Bloc"
     client: false
-  debtcollective_redirect_url_after_signup:
+  debtcollective_after_signup_redirect_url:
     default: "https://debtcollective.org/hub"
     client: true
diff --git a/lib/extensions/invites_controller.rb b/lib/extensions/invites_controller.rb
@@ -31,7 +31,7 @@ def perform_accept_invitation
 
             # If this is a new user or first login, redirect
             # we only set it if topic invite is nil
-            redirect_to = SiteSetting.debtcollective_redirect_url_after_signup
+            redirect_to = SiteSetting.debtcollective_after_signup_redirect_url
             if (user.new_user? || !user.seen_before?) && redirect_to.present? && topic.blank?
               response[:redirect_to] = redirect_to
             end

diff --git a/lib/extensions/session_controller.rb b/lib/extensions/session_controller.rb
@@ -82,7 +82,7 @@ def email_login
 
           response = { success: 'OK' }
 
-          redirect_to = SiteSetting.debtcollective_redirect_url_after_signup
+          redirect_to = SiteSetting.debtcollective_after_signup_redirect_url
           response[:redirect_to] = redirect_to if redirect_to.present? && !user.seen_before?
 
           return render json: response

diff --git a/spec/requests/session_controller_spec.rb b/spec/requests/session_controller_spec.rb
@@ -41,7 +41,7 @@
     it "redirects to after_signup_path if it's the first login" do
       user = Fabricate(:user)
       email_token = user.email_tokens.create!(email: user.email)
-      SiteSetting.debtcollective_redirect_url_after_signup = "https://example.com"
+      SiteSetting.debtcollective_after_signup_redirect_url = "https://example.com"
 
       post "/session/email-login/#{email_token.token}", headers: { "ACCEPT" => "application/json" }
       json = JSON.parse(response.body)