social icon indicating copy to clipboard operation
social copied to clipboard

after installing mail debrand in odoo 15 community edition digest email not display properly showing HTML code instead of message

Open Zebrando opened this issue 3 years ago • 9 comments

Hi,

I have an issue after installing mail debrand in odoo 15 community edition digest email does not display properly, showing HTML code instead of the message, is there any solution for this? below is the attached screenshot. image

Zebrando avatar Nov 28 '22 06:11 Zebrando

I'm seeing same thing, not reliable to have the module installed

arturoovf avatar Feb 02 '23 19:02 arturoovf

I ended up uninstalling and modifying the templates myself

arturoovf avatar Mar 15 '23 19:03 arturoovf

Had the same issue, should be solved in the meantime with #939

dhx avatar Jun 13 '23 13:06 dhx

@dhx @rven @pedrobaeza @joao-p-marques this is back again at 15.0 branch. Tested in runboat. My workaround involved to change: https://github.com/OCA/social/blob/938f9ec7bda6d7960e5e3bc38fc39508bd9d3cad/mail_debrand/models/mail_render_mixin.py#L54-L56 To: value = Markup(html.tostring( tree, pretty_print=True, method="html", encoding="unicode" ))

rrebollo avatar Nov 02 '23 02:11 rrebollo

@dhx @rven @pedrobaeza @joao-p-marques this is back again at 15.0 branch. Tested in runboat. My workaround involved to change:

https://github.com/OCA/social/blob/938f9ec7bda6d7960e5e3bc38fc39508bd9d3cad/mail_debrand/models/mail_render_mixin.py#L54-L56

To: value = Markup(html.tostring( tree, pretty_print=True, method="html", encoding="unicode" ))

@rrebollo this should be in a pull request to be merged, as today this is not been solved in the latest version of mail_debrand module"version": "15.0.1.2.4" , does there is a reason why i hasn't been done yet ?

Garibaldy avatar Mar 17 '24 16:03 Garibaldy

@Garibaldy suggested workaround doesn't fix the issue either for some mail templates.Ex: mail template to invite new users is broken by the attend to replace odoo links using regex. I ended up modifying the mail templates.

rrebollo avatar Mar 17 '24 16:03 rrebollo

@Garibaldy Possible alternative to replace the templates: https://github.com/OCA/social/tree/15.0/mail_layout_force ( PR for 16.0: https://github.com/OCA/social/pull/1069 )

dhx avatar Mar 17 '24 17:03 dhx

@Garibaldy suggested workaround doesn't fix the issue either for some mail templates.Ex: mail template to invite new users is broken by the attend to replace odoo links using regex. I ended up modifying the mail templates.

i double checked twice all templates and it seems to be working just right with your workaround,

maybe because a pull request that was made after your suggestion like this one metioned before https://github.com/OCA/social/pull/939 , fixed most of the things except digest

Garibaldy avatar Mar 18 '24 02:03 Garibaldy

There hasn't been any activity on this issue in the past 6 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this issue to never become stale, please ask a PSC member to apply the "no stale" label.

github-actions[bot] avatar Sep 15 '24 12:09 github-actions[bot]