Magento2: Magento 2.1 email logo image function does not work

Created on 29 Jun 2016  Â·  73Comments  Â·  Source: magento/magento2

Steps to reproduce

  1. Magento 2.1
  2. Upload Email logo in path Content -> Configuration -> Transactional Email -> Logo Image
  3. Save

    Expected result

  4. New Email Logo should show in New Order (ALL) Emails

    Actual result

  5. Default Magento Logo is used in email

  6. Customized logo does not show
  7. Logo Image URL Broken in preview under path Content -> Configuration -> Transactional Email -> Logo Image
Format is not valid bug report

Most helpful comment

Why is this issue closed? This issue is not solved in 2.1.3.

All 73 comments

This issue was fixed in MAGETWO-53010 and delivered to develop branch

how do i find the file to update? i just copied the missing image location and added the folder there? but i would like to fix the issue

Cheers sorted, Thanks for the link. appreciated :)
You said "This issue was fixed in MAGETWO-53010"

I didn't know how to find the fix "MAGETWO-53010" i still dont . i see lots of people stating this was fixed in MAGETWO-XXXX But i have no idea how to find it?

Regards

When issue is closed link should be attached, but since this one is open you can search MAGETWO-XXXX in pull requests and then find attached commits in it.

Ok thanks for your help, appreciated.
Regards

@Nerogee the issue has been fixed and delivered to mainline. Please see 23103c0, 2f32aed and cb603ff

What should we do about this in the mean time ? Abstract class and protected methods. This rules out a temporary preference and plugin. Anyone have a better idea than a symlink ?

Hi there

this issue was supposed to be patched in the nearest release, but the 2.1.1 still has the issue and it was released yesterday. not cool

Just to confirm what samc2810 wrote, the issue has not been fixed in 2.1.1.

Can someone guide me how to fix this? I have 2.1.1. Everyone is sending Luma logos in their emails? ARe there not that many people using 2.1? I don't get it. Why isn't this not considered a major issue?

Can confirm the bug, dirty/temporary way to solve is:
1) Upload the image
2) Check the file path in preview box
3) Copy that file to pub/media/email/logo/stores/1/ (1 can be other number, depending on the store id)
4) Save the configuration

I have spend the entire day trying to get this to work without any success. I am on ver 2.1.1 brand new installation this functionality worked with previous version, I am confused to why mess with something that worked.
anyway - I upload the image in the backend since the upload doesnt actuly upload the file, I click to view the file path i see - pub/media/email/logo/stores/1/logo_email_2.png - I manually upload the file logo_email_2.png to the following directory.
I have also uploaded this same fie in - /vendor/magento/module-email/view/frontend/web

New order e-mail is still not showing the image (see attached). when i follow the link to the image from the email - I have the correct image it shows the image i uploaded.
noiamge

I have removed all the email logo images from the backend cleared all the cache left it as default luma logo and it shows up correctly for luma logo in the email....what i cannot figure out where is this Luma email logo coming from, when i click to find the path in the email, it takes me to my personal logo. i have cleared all the cache also the static folders about 100 times - what a nightmare. can some one help me with this

Thank you

@nht48823 Seem that Google will don't show image without https. We should inspect the image link, and go to this image, if it exists, your logo image is correct.

Is anyone able to confirm if a fix for this has been included the new 2.1.2 release?

@timbaker1991, unfortunately no.

@timbaker1991 Well, it definitely had to be included in Release 2.1.2 ... Still waiting for the fix!

It looks like it is still not repaired in 2.1.2

one quick and dirty way is to edit vendor/magento/module-email/view/frontend/email/header.html line 50, replace the {{xxx}} by the src of your image for example : https://www.example.com/pub/media/logo/stores/1/logo.png

Is somebody still working on this because it is still not working for me. I am only sending out emails the Luma logo. How do others work around this?

thanks for any help

confirmed still broken on 2.1.2

Hi,
I have same issue. Any new?

same problem here on 2.1.0, workin on Store View scope

what i notice is
logo is uploaded into pub/media/tmp/design/file/
logo is saved into pub/media/logo/stores/1/
logo is read -in admin backend- from pub/media/email/logo/stores/1/
logo is read -in email- from
pub/static/frontend/Vendor/Module/en_US/Magento_Email/
(default Magento logo is used, because custom logo is not found)

solved copying the logo into the manually created folder pub/media/email/logo/stores/1/

so the problem is when you save the logo, and it is uploaded to the wrong directory

didn't try on 2.1.2, but will do and report soon

i can confirm that in M2.1.2 the logo is still saved under pub/media/logo/stores/1/ instead of pub/media/email/logo/stores/1/ wich is the path that the logo is read from, both in email and in backend

@andreaschiffo
Thanks for your research and feedback. I use M2.1.2 and I fully understand what your are saying but the image will not show.
I don't have a stores folder but it is websites or default. There rest of the path is the same. Is stores required?
I use a .svg or a .png file. Do you give image size parameters in the backend?

It is an annoying little issue that makes every transactional email that I am sending look stupid. And it makes me look for the tiniest detail.

Thanks to anyone who is trying to solve this in a reasonable way in the backend.

@olafc100 understand your feelings ;)
i should check your stores&websites config to understand why you dont have the stores folder
i have the stores folder because i'm editing the default store view configuration

cattura

as a general hint look at the broken image url, where magento is expecting to find the logo, and put your custom image there

cattura2

then remove all the content of pub/static folder, clear the cache and redeploy static content (bin/magento setup:static-content:deploy)

@andreaschiffo thanks for keeping up the conversation and descriptive images.

It solved a part of the problem:
the stores and websites are the difference between editing only on the global level. So the top level on your first images. When I edit on your level (store view) it indeed stores (or is trying to store) the logo in a stores directory. That is what I see when I check the broken link as you are showing in your second image. Magento backend even attached an _1 to the image name. Don't know why. But after creating the directory structure in the backend it remembers and shows the logo. Unfortunately only in the backend and still not in the emails. Perhaps .svg creates problems in email programs. I will try with different file types. Thanks again for the help.

@andreaschiffo Yes, logo is showing! Sweet success.

I did flush the cache storage also this time perhaps that helped.
The _1.svg addition was probably because the image already existed in the directory.

Thanks again for your time and effort.

Hello Folks,

I am still facing this issue...has there been a fix.

@scudrock seems like it hasnt, http://devdocs.magento.com/guides/v2.1/release-notes/ReleaseNotes2.1.3CE.html
i think they fixed only some kind of issue related to email templates, otherwise they would have closen this bug
i am not able to test 2.1.3 right now, so you may try creating manually the folder pub/media/email/logo/stores/1/ and copying the logo into that path

Why is this issue closed? This issue is not solved in 2.1.3.

I can't find a solution to this. /pub/media/email/logo/default/ There is no email folder in the media folder. I even created the folders manually and put the image in there. Still not working. Please help

@nbjohan think its open because the issue is being tracked in #6275

@mattlaltman check file and directories permissions, clean magento cache, clean the pub/static folder, try even redeploying static content;
just like i was stating, as a general hint, look at the broken image url, where magento is expecting to find the logo, and put your custom image there
for my 2.1.2 version it worked like that, dont know if youre facing a different magento behaviour, don't know if it's related on your stores/websites config or whatsoever.

In case anyone needs help, anyway, i think it's better to use magento.stackexchange.com; here's just for bug tracking

Why is this issue closed? This issue is not solved in 2.1.4.

Still not solved! Why can't this be fixed? Issue is active since a while.

@sevos1984 please reopen

Please reopen this issue and add to 2.1.6... (2.1.5 is security update)

This is very annoying bug i still faced in 2.1.4 .

Here is my hack:
1) Override header.html in your theme and replace following logo url line.

src='{{store url=""}}pub/media/email/logo/stores/{{var store.getId()}}/logo_email.png'

2) Replace logo files in /pub accordingly.

No need to edit any file or any mess-up with codes.
Just do these simple things and all your email logo things will be fixed.

Step 1.
Rename your logo to "logo_email.png" in your computer.

Step 2.
Just search for "logo_email.png" in Magento installed directory in your server.
--in my case it is in "public_html" directory.
ice_screenshot_20170225-133635

Step 3.
Just replace these above two ".png"(inside black rectangles) files with your ".png" file and "Flush cache" & "Flush static files cache" in Magento from your admin_panel and give your email template a preview & see its working.

This is not working when using multistore...

Op za 25 feb. 2017 om 09:02 schreef Sohail-Ansari notifications@github.com

No need to edit any file or any mess-up with codes.
Just do these simple things and all your email logo things will be fixed.

Step 1.
Rename your logo to "logo_email.png" in your computer.

Step 2.
Just search for "logo_email.png" in Magento installed directory in your
server.
--in my case it is in "public_html" directory.
[image: ice_screenshot_20170225-133635]
https://cloud.githubusercontent.com/assets/26020817/23329420/04f3783e-fb5e-11e6-8716-df0fa13b5d9d.png

Step 3.
Just replace these above two ".png"(inside black rectangles) files with
your ".png" file and "Flush cache" in Magento from your admin_panel and
give your email template a preview & see its working.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/magento/magento2/issues/5352#issuecomment-282468405,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXGrS_GwzuwJohgp_L_e7TWFqhhtQsTMks5rf9_1gaJpZM4JBGmP
.

>

Met vriendelijke groet, Kind regards, Henk de Gier Algemeen Directeur
Certified Magento Frontend Developer 's werelds snelste Magento platform
Hypershop B.V. Laan van Vredenoord 29a 2289 DA Rijswijk www.hypershop.nl

Copying the logo files to /pub/email/email/ seemed to work for me. How can they leave this unfixed for so long?

Bump.

Just to add an update, in CE 2.1.5 it is not fixed.
Copying the missing folder, or using a symlink does appear to 'solve' the problem.

@sevos1984 please reopen this issue

Confirmed issue in 2.1.5

And still an issue in EE 2.1.6 :cry:

Saving your logo to Magento_Email/web/logo_email.png in your theme-folder will work (remember to deploy static files).

Ridiculous that this hasn't been resolved yet.

And don't forget to compile again before deploying static files. Just learned the hard way -

running this:
php bin/magento setup:static-content:deploy

without running this first:
php bin/magento setup:di:compile

Results in an error that'll break your site. GottaLoveIt...
*If there's a better way, please advise.

..spoke too soon. Overwriting "logo_email.png" in theme folders & recompiling didn't even work.
Wound up reading this: https://magento.stackexchange.com/questions/128237/magento-2-1-fails-to-upload-transactional-email-logo-from-admin

And decided to just create the directories manually (/pub/media/logo/stores/1/email_logo.png)
1.5hrs is enough time wasted on such a SIMPLE thing, GoodGods

ps. This is all per upgrading from 2.1.0 to 2.1.6.

I have 2.1.7 and this problem is still there... :(

I suspect it won't be until 2.2.x as there are other bugs that are also not being deployed. I hope that's the case anyway.

@timbaker1991: I backported the fix from the develop branch to the 2.1-develop branch over here:
https://github.com/magento/magento2/pull/9590

So it should get included in the next minor release (2.1.8 hopefully).

still not fixed at all in 2.1.6 - one year later now and magento does not get it at all.

@magento-team what the problem here?

@andidhouse: have you read my comment above? It will be fixed in (probably/hopefully) the next 2.1.x release.

@hostep - no i did not read that! There are about 30 bugs reported at GitHub according to the same email logo topic and you simply can not read all of them.

But to be honest i really miss timelines and quick fixes of the magento team. This bug is now over a year reported and not solved. Now its gonna be solved "maybe" in 2.1.7 or 8... ok when should that be and what is "maybe"?

I think too many people spent too many time on reporting and fixing bugs for m2. The way more simple solution in my opinion would be to introduce a system with not so many bugs ;-).

But thanks very much for the info also it is not really of value for us users with no timeline at all.

confirmed still broken on magento2 2.1.6

Same on 2.1.7. Anyway, just copied the folder to the right place.

guys, it will be fixed .. one day, yes, either in 2.1.xx , or m3 or m4 :)
I found a dozen of 1-year-old bugs that have been fixed, but they are still in ..Development branch , people never released them
Very bad management

I am watching several issues which have a very similar theme... just have to hope that they are working on a mega release at some point

On 11 Jul 2017, at 22:14, congaframe notifications@github.com wrote:

guys, it will be fixed .. one day, yes, either in 2.1.xx , or m3 or m4 :)
I found a dozen of 1-year-old bugs that have been fixed, but they are still in ..Development branch , people never released them
Very bad management

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

follow

Internal ticket to track issue progress: MAGETWO-69062

Still having this issue running a live 2.1.7 multi-store. Hope it gets fixed soon. Hate messing with code.

@commalex try upgrading to 2.1.8 which was released yesterday (see http://devdocs.magento.com/guides/v2.1/release-notes/ReleaseNotes2.1.8EE.html#general).

Still having the issue after upgrading to 2.1.8
The image now shows fine in admin panel, however mail still shows default Magento logo, not the custom logo.
The custom theme we are using does not overwrite the email templates.
Anyone else facing similar issue? Any advise?
Thank you.

Is this really still an issue after being reported over a year ago? Bug still appears to be present in both 2.1.7 and 2.1.8

@apexdivision @carb0nknight issue seems to be fixed according to comments in https://github.com/magento/magento2/commit/23103c03c527fe1ed3e1a3be691436818b7963fe#commitcomment-23995349. It is always better to report a new issue describing all necessary details referring to a closed one than comment on a closed issue.

I have a quick solution to do this and it will done by changing core files.Open magento vendor/magento/module_email/view/frontend/email header.phtml file.In this file change the img src by adding your logo img hardcode around line 50.

@Anees-Ahmad That's not a solution, it's a workaround, one that will be required to be repeated anytime you pull changes to the core files or update.

This Issue is also present in magento 2.3 version

email_custom

Hi @surbhi-ranosys. Thank you for working on this issue.
In order to make sure that issue has enough information and ready for development, please read and check the following instruction: :point_down:

  • [ ] 1. Verify that issue has all the required information. (Preconditions, Steps to reproduce, Expected result, Actual result).
    DetailsIf the issue has a valid description, the label Issue: Format is valid will be added to the issue automatically. Please, edit issue description if needed, until label Issue: Format is valid appears.
  • [ ] 2. Verify that issue has a meaningful description and provides enough information to reproduce the issue. If the report is valid, add Issue: Clear Description label to the issue by yourself.

  • [ ] 3. Add Component: XXXXX label(s) to the ticket, indicating the components it may be related to.

  • [ ] 4. Verify that the issue is reproducible on 2.3-develop branch

    Details- Add the comment @magento-engcom-team give me 2.3-develop instance to deploy test instance on Magento infrastructure.
    - If the issue is reproducible on 2.3-develop branch, please, add the label Reproduced on 2.3.x.
    - If the issue is not reproducible, add your comment that issue is not reproducible and close the issue and _stop verification process here_!

  • [ ] 5. Verify that the issue is reproducible on 2.2-develop branch.

    Details- Add the comment @magento-engcom-team give me 2.2-develop instance to deploy test instance on Magento infrastructure.
    - If the issue is reproducible on 2.2-develop branch, please add the label Reproduced on 2.2.x


Hi @engcom-Charlie. Thank you for working on this issue.
In order to make sure that issue has enough information and ready for development, please read and check the following instruction: :point_down:

  • [ ] 1. Verify that issue has all the required information. (Preconditions, Steps to reproduce, Expected result, Actual result).
    DetailsIf the issue has a valid description, the label Issue: Format is valid will be added to the issue automatically. Please, edit issue description if needed, until label Issue: Format is valid appears.
  • [ ] 2. Verify that issue has a meaningful description and provides enough information to reproduce the issue. If the report is valid, add Issue: Clear Description label to the issue by yourself.

  • [ ] 3. Add Component: XXXXX label(s) to the ticket, indicating the components it may be related to.

  • [ ] 4. Verify that the issue is reproducible on 2.3-develop branch

    Details- Add the comment @magento give me 2.3-develop instance to deploy test instance on Magento infrastructure.
    - If the issue is reproducible on 2.3-develop branch, please, add the label Reproduced on 2.3.x.
    - If the issue is not reproducible, add your comment that issue is not reproducible and close the issue and _stop verification process here_!

  • [ ] 5. Add label Issue: Confirmed once verification is complete.

  • [ ] 6. Make sure that automatic system confirms that report has been added to the backlog.

Hello @Nerogee
I can't reproduce this issue on fresh Magento 2.3-develop.
So i close it.
Thank for your report!

Was this page helpful?
0 / 5 - 0 ratings