diff --git a/frappe/email/doctype/newsletter/newsletter.py b/frappe/email/doctype/newsletter/newsletter.py
old mode 100755
new mode 100644
index a118240488..7c0e2dfe87
--- a/frappe/email/doctype/newsletter/newsletter.py
+++ b/frappe/email/doctype/newsletter/newsletter.py
@@ -159,11 +159,10 @@ class Newsletter(WebsiteGenerator):
 	def send_newsletter(self, emails: List[str]):
 		"""Trigger email generation for `emails` and add it in Email Queue.
 		"""
-		# TODO: get rid of this maybe?
-		message = self.get_message()
 		attachments = self.get_newsletter_attachments()
 		sender = self.send_from or frappe.utils.get_formatted_email(self.owner)
-		args = {"message": message, "name": self.name}
+		args = self.as_dict()
+		args["message"] = self.get_message()
 
 		is_auto_commit_set = bool(frappe.db.auto_commit_on_many_writes)
 		frappe.db.auto_commit_on_many_writes = not frappe.flags.in_test
@@ -172,7 +171,6 @@ class Newsletter(WebsiteGenerator):
 			subject=self.subject,
 			sender=sender,
 			recipients=emails,
-			message=message,
 			attachments=attachments,
 			template="newsletter",
 			add_unsubscribe_link=self.send_unsubscribe_link,
diff --git a/frappe/templates/emails/newsletter.html b/frappe/templates/emails/newsletter.html
index a3afb906cf..051840ef69 100644
--- a/frappe/templates/emails/newsletter.html
+++ b/frappe/templates/emails/newsletter.html
@@ -3,8 +3,11 @@
         {{ message }}
     </div>
 </div>
+
+{% if published and send_webview_link %}
 <div style="font-size: 12px; line-height: 20px;">
     <div>
         Open in <a style="color: #687178; text-decoration: underline;" href="/newsletters/{{ name }}" target="_blank">web</a>
     </div>
-</div>
\ No newline at end of file
+</div>
+{% endif %}
\ No newline at end of file