search

responsiv / campaign-plugin

[PREMIUM] Send professional campaign messages to your subscribers.
http://octobercms.com/plugin/responsiv-campaign
5 stars 2 forks source link

Mail campaign cropped when content is too long #60

Closed franadr closed 3 years ago

franadr commented 4 years ago

Hello, might there be a size limit for the content of a campaign imposed by the plug-in ? In fact I experienced some cropping on the bottom of the preview and the received mail when the text was more than a certain size. (I didn't measured it precisely)

The template proved to be well functioning when using it as a stand-alone html page ( outside of the campaign manager/October CMS context) and when the text content wasn't excessively long.

What the bottom should look like:

When the text is too long : (Bottom is cropped)

LukeTowers commented 4 years ago

Is this happening in all the mail viewers or just a select few?

LukeTowers commented 4 years ago

If you click on "View Original" in your mail client does it have the HTML for the entire message?

franadr commented 4 years ago

All the viewer yes (web client and desktop client) The "view original" version is also cropped. The html doesn't include the missing part.

LukeTowers commented 4 years ago

@franadr if you look in the database tables for the message, is all of the HTML present there? Currently the table uses TEXT fields for storing the message content, which has a limit of 65000 characters. I've bumped it up to MEDIUMTEXT in the current codebase because of a similar issue I was having but that hasn't been released yet.

franadr commented 4 years ago

Thank you very much for the info it indeed explains why the last part was missing (on the content_html attribute the corresponding message is at TEXT size limit (64KB) event though content attribute isn't (37KB) and includes all the html ). As from your experience is it safe to pass it to MEDIUMTEXT in the responsiv_campaign_messages table directly or should it be done programmatically ? As there are 2 column that seems to contain the HTML content : content and content_html.

LukeTowers commented 4 years ago

@franadr for now you should be able to manually edit the three columns that use TEXT to use MEDIUMTEXT (including the data one), in a future update there will be a migration that does it for you automatically but it shouldn't break anything if you manually alter those columns in the meantime.

franadr commented 4 years ago

Made the changes and the mail content is not cropped any more, thank you!