Skip to content

Commit 0c39163

Browse files
committed
Remove flash message for newsletter create
Because redirect page already implements a success message
1 parent 126d925 commit 0c39163

File tree

2 files changed

+1
-5
lines changed

2 files changed

+1
-5
lines changed

app/controllers/users/newsletter_subscriptions_controller.rb

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,7 @@ def create
4444
# TODO: Send already subscribed email
4545
end
4646

47-
redirect_to users_newsletter_subscription_path(@newsletter_subscription),
48-
notice: "Welcome to Joy of Rails! Please check your email for confirmation instructions"
47+
redirect_to users_newsletter_subscription_path(@newsletter_subscription)
4948
end
5049

5150
def subscribe

spec/requests/users/newsletter_subscriptions_spec.rb

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,6 @@
6868
}.to change(User, :count).by(1)
6969

7070
expect(response).to redirect_to(users_newsletter_subscription_path(User.last.newsletter_subscription))
71-
expect(flash[:notice]).to eq("Welcome to Joy of Rails! Please check your email for confirmation instructions")
7271

7372
perform_enqueued_jobs_and_subsequently_enqueued_jobs
7473

@@ -90,7 +89,6 @@
9089
}.not_to change(NewsletterSubscription, :count)
9190

9291
expect(response).to redirect_to(users_newsletter_subscription_path(User.last.newsletter_subscription))
93-
expect(flash[:notice]).to eq("Welcome to Joy of Rails! Please check your email for confirmation instructions")
9492

9593
expect(user.reload.newsletter_subscription).to be_present
9694
# assert "already subscribed" email sent
@@ -105,7 +103,6 @@
105103
}.to change(NewsletterSubscription, :count).by(1)
106104

107105
expect(response).to redirect_to(users_newsletter_subscription_path(User.last.newsletter_subscription))
108-
expect(flash[:notice]).to eq("Welcome to Joy of Rails! Please check your email for confirmation instructions")
109106

110107
expect(user.reload.newsletter_subscription).to be_present
111108
end

0 commit comments

Comments
 (0)