Skip to content
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

Error "Icon must be an image." when attempting to create some appliction Launcers #1179

Closed
mattbaker109 opened this issue Jun 23, 2023 · 24 comments · Fixed by #1222
Closed

Comments

@mattbaker109
Copy link

OS: Debain 12 and Ubuntu 22.04. PHP v7.4.33. Heimdall version: 2.5.7

After a fresh install, attempt to add a new Launcher for Proxmox or Syncthing. Once hitting the save button get an error: "Icon must be an image." and the launcher will not save.

@mattbaker109
Copy link
Author

Revert to version 2.5.6 and application is working as expected.

@Anarch157a
Copy link

My workaround is to right-click on the displayed icon, save the image then re-upload it. But it's annoying.

@LinuxServer-CI
Copy link

This issue has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions.

@allw1994
Copy link

Issue for me too, best I can tell the Store@ItemController method is receiving the correct url and will work if the image is a .png but if it is .svg it does not work.

@GJSchaller
Copy link

Issue for me as well - Heimdall 2.5.7, Rocky Linux 9.2, Apache 2. Issue happens with both links to external websites, and pre-defined Apps.

@allw1994
Copy link

Anyone come across a solution to this?

Starting to annoy me the number of steps to get a single link to work and the fact the integrations don't work.

@LinuxServer-CI
Copy link

This issue has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions.

@allw1994
Copy link

allw1994 commented Oct 17, 2023

Still waiting on PR merge #1222

@ovhelemann
Copy link

I have the same issue on version 2.5.7 (docker)

@abhisheknair
Copy link

Facing same issue with v2.5.7 (docker)

@yotis1982
Copy link

I've noticed that if I add a new app and it has an image which is .png type, all is good. If the image is .svg, I need to download it and upload like mentioned above.

@jonaavanza
Copy link

I have the same issue with the latest docker image

@pmontgo33
Copy link

Also having this issue running version 2.5.7 in a Debain Bookworm LXC on Proxmox.

@Halfang
Copy link

Halfang commented Oct 30, 2023

Having the same issue with docker 2.5.7
Linuxserver.io version:- v2.5.7-ls237 Build-date:- 2023-10-27T18:31:16+00:00

@konradmazurczak
Copy link

same here
BUILD_DATE=2023-10-27T18:31:16+00:00 VERSION=v2.5.7-ls237 HEIMDALL_RELEASE=v2.5.7

@ADIX7
Copy link

ADIX7 commented Nov 1, 2023

Same here

@meminens
Copy link

meminens commented Nov 2, 2023

Same here. Downloading the icon and re-uploading appears to be working.

@arnovdk
Copy link

arnovdk commented Nov 2, 2023

All, I would say there doesn't seem necessary to further post anymore "Me 2!" messages. We all see that the problem exists and that an effective work-around has been suggested.

I subscribed to this thread so that I might stay informed about any progress, but getting notifications for these messages really isn't adding or helping. If you haven't considered it, you could help out by investigating the bug and discovering a possible solution. But as is the case with me (and possibly the maintainers of this project), time is scarce and also maybe like me, maybe you lack the skills to do so. So for me the work-around helps for now, until the solution presents itself. 🙂

Thank you for your consideration!

@allw1994
Copy link

allw1994 commented Nov 3, 2023

The work arounds don’t give full functionality though.

Downloading and reuploading the image means you can’t use the integrations.

Changing the controller as per PR #1222 breaks other things.

I’ve been looking at this myself and without the PR and further changes we are stuck but they still haven’t approved the PR or made comment on suggestions as to how to fix any of the bugs in the PR.

@ZoltrixGFC
Copy link

Also have seen the same issue on version 2.5.7 for SABnzbd.

@causalmask
Copy link

If it's going to take months to approve a fix, why wasn't 2.5.7 rolled back to 2.5.6 months ago? Everyone attempting to install since 2.5.7 was released has had to manually roll back.

@ElDubsNZ
Copy link

Getting the same issue as others on this.

@LinuxServer-CI LinuxServer-CI moved this from Non-Docker Issues to Done in Issue & PR Tracker Nov 11, 2023
@Stoony1
Copy link

Stoony1 commented Nov 11, 2023

Same Problem here i will create a Bookmark for Pihole an hes say "Icon must be an image." and i dont can Create it..

@j0nnymoe
Copy link
Member

The fix has been merged, new release has been cut, wait for the updated docker image to be built.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.