From af8978bd5c7679e338313e79aff41af2b7c24828 Mon Sep 17 00:00:00 2001 From: moveson Date: Fri, 12 Jul 2024 15:40:00 -0600 Subject: [PATCH 1/2] Replace SMS subscription button with out-of-service message --- app/views/efforts/_subscription_buttons.html.erb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/views/efforts/_subscription_buttons.html.erb b/app/views/efforts/_subscription_buttons.html.erb index 6b3e32456..2c870ca1a 100644 --- a/app/views/efforts/_subscription_buttons.html.erb +++ b/app/views/efforts/_subscription_buttons.html.erb @@ -6,6 +6,6 @@
<%= render "subscriptions/subscription_button", subscribable: effort, protocol: :email %> - <%= render "subscriptions/subscription_button", subscribable: effort, protocol: :sms %> + SMS temporarily out of service.
Please use email.
From 8ecf9a8c62e889badce003835bdb16fe9565c56f Mon Sep 17 00:00:00 2001 From: moveson Date: Fri, 12 Jul 2024 15:48:18 -0600 Subject: [PATCH 2/2] Suspend SMS subscription tests --- .../subscriptions/subscribe_to_effort_notifications_spec.rb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/spec/system/subscriptions/subscribe_to_effort_notifications_spec.rb b/spec/system/subscriptions/subscribe_to_effort_notifications_spec.rb index 7c8cf2aaf..92258fe32 100644 --- a/spec/system/subscriptions/subscribe_to_effort_notifications_spec.rb +++ b/spec/system/subscriptions/subscribe_to_effort_notifications_spec.rb @@ -10,7 +10,7 @@ before { effort.update!(topic_resource_key: "anything") } - scenario "The user is not logged in and subscribes to sms" do + xscenario "The user is not logged in and subscribes to sms" do visit_page within("##{dom_id(effort, :sms)}") { click_button("sms") } @@ -36,7 +36,7 @@ expect(page).to have_content("You have subscribed to email notifications for #{effort.full_name}.") end - scenario "The user is logged in and subscribes to sms without a phone number" do + xscenario "The user is logged in and subscribes to sms without a phone number" do login_as user, scope: :user visit_page @@ -46,7 +46,7 @@ expect(page).to have_content("Please add a mobile phone number to receive sms text notifications.") end - scenario "The user is logged in and subscribes to sms with a phone number" do + xscenario "The user is logged in and subscribes to sms with a phone number" do user.update_columns(phone: "1234567890") login_as user, scope: :user visit_page