Apr 28, 2021 · Apr 28, 2021 · Apr 28, 2021 · Aug 15, 2021
diff --git a/google-apps-script.js b/google-apps-script.js
Original file line number	Diff line number	Diff line change
Expand Up		@@ -59,7 +59,8 @@ function doPost(e) {
		if (sendEmailTo) {
		MailApp.sendEmail({
		to: String(sendEmailTo),
	subject: "Contact form submitted",
	name: String(e.parameters.formGoogleEmailSender) \|\| "Contact Form",
	subject: String(e.parameters.formGoogleEmailSubject) \|\| "Contact form submitted",
Copy link Collaborator @mckennapsean mckennapsean Aug 15, 2021 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. I believe these `\|\| ...` checks won't get hit in all cases. If the parameters are null or undefined, (like if someone just updates the backend script here or doesn't give these parameters), then you would end up with `String(undefined) = 'undefined'` which is not falsey. It may be worth setting the name / subject above and consider using `mailData` and/or `.toString()` when the field is present/truthy. nelsonic reacted with thumbs up emoji
		// replyTo: String(mailData.email), // This is optional and reliant on your form actually collecting a field named `email`
		htmlBody: formatMailBody(mailData, dataOrder)
		});
Expand Down