From 60b3e6777fef7969a0dcaf4f86a977e07fdaa732 Mon Sep 17 00:00:00 2001
From: Henry Oswald <henry.oswald@sharelatex.com>
Date: Tue, 28 Aug 2018 13:34:53 +0100
Subject: [PATCH] always set status_if_new

---
 .../app/coffee/Features/Newsletter/NewsletterManager.coffee  | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/services/web/app/coffee/Features/Newsletter/NewsletterManager.coffee b/services/web/app/coffee/Features/Newsletter/NewsletterManager.coffee
index f64b8b7d63..dba4919496 100644
--- a/services/web/app/coffee/Features/Newsletter/NewsletterManager.coffee
+++ b/services/web/app/coffee/Features/Newsletter/NewsletterManager.coffee
@@ -59,16 +59,17 @@ hashEmail = (email)->
 
 buildOptions = (user, is_subscribed)->
 	subscriber_hash = hashEmail(user.email)
+	status = if is_subscribed then "subscribed" else "unsubscribed"
 	opts =
 		method: "PUT"
 		path: "/lists/#{Settings.mailchimp?.list_id}/members/#{subscriber_hash}"
 		body:
 			email_address:user.email
+			status_if_new: status
 				
-	status = if is_subscribed then "subscribed" else "unsubscribed"
+	#only set status if we explictly want to set it
 	if is_subscribed?
 		opts.body.status = status
-		opts.body.status_if_new = status
 
 	if user._id?
 		opts.body.merge_fields =