-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NV 546 fix: clipped blocked html #722
NV 546 fix: clipped blocked html #722
Conversation
NV-546 Email template is clipped due to large size on G-mail
When sending a simple email message the email is clipped by google due to email size. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -203,7 +203,7 @@ export class SendMessageEmail extends SendMessageType { | |||
const messageVariables = contentService.buildMessageVariables(command.payload, subscriber); | |||
const content: IEmailBlock[] = [...emailChannel.template.content] as IEmailBlock[]; | |||
for (const block of content) { | |||
block.content = contentService.replaceVariables(block.content, messageVariables); | |||
block.content = contentService.replaceVariables(block.content, messageVariables).trim(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe worth adding a comment here about why the .trim
is important :P @djabarovgeorge
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably it worth the commit :)
Will do
@scopsy added note, let me know if you think its understandable |
@djabarovgeorge Looks good! Let's merge it 🚀 |
No description provided.