Vauxoo / odoo

Fork of Odoo (formerly OpenERP). [This project is not publically mantained just born for internal usage with some little patches] go to official repository on github.com/odoo/odoo
https://www.odoo.com
Other
9 stars 9 forks source link

Image of email template does not show in email message #101

Open ysantiago opened 8 years ago

ysantiago commented 8 years ago

When add an image in email template with its code base64, this does not show propertly in the email message.

skitch 12 1
gquilarque commented 8 years ago

Issue moved to Vauxoo/yoytec #1335 via ZenHub.io

friedi1 commented 8 years ago

@ysantiago I can't see the logo in the templates received today in the test, this problem got resolved?

moylop260 commented 8 years ago

@friedi1 FYI The issue is confirmed from odoo/odoo and is WIP: https://github.com/odoo/odoo/issues/12050

We are waiting the fix to apply it

gquilarque commented 8 years ago

Hi @ysantiago @moylop260

Do you know anything about this issue?

I did a POC today and I got the following email without logo:

image

Best Regards

moylop260 commented 8 years ago

We have the confirmed issue https://github.com/odoo/odoo/issues/12050 But odoo don't send us the fix yet. I will ask them about.

IMHO we could use link instead of image attached for while.

What do you think?

moylop260 commented 8 years ago

Bad news we have the issue odoo.com#680023

But we have a bad news, now is cancelled without a explanation: image

gquilarque commented 8 years ago

Hi @moylop260

Thanks for answering!

Could you give me an example of how the logo would look like in the email with a link instead of image?

moylop260 commented 8 years ago

@gquilarque Sure

@ysantiago Could you create the new PR with link instead of attach?

ysantiago commented 8 years ago

Hello @moylop260 Remember that the image with a link does not work because it does not show in email body when it was sent. Here PR where it was generated the need to attach image.

moylop260 commented 8 years ago

You are right! The image is not showed from server with a block url configured that is the reason of "why" we need a attach. But If we don't have the attach now, the url could fix for while.

ysantiago commented 8 years ago

@moylop260 roger that