-
-
Notifications
You must be signed in to change notification settings - Fork 93
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
[Bug] Custom Covers not saved #1169
Comments
That's a new one, usually it needs to delete the old image before uploading a new one, but it can't seem to find it. Between this and your other issue, there might be something wonky with your setup. Can you verify that the perms for the |
What should the permissions be? My setup is like this:
Everything is owned by root. Maybe that's not a good idea? |
No that should be fine, though libreelec is an interesting choice! Can you post your docker compose here, hiding any sensitive info/keys? |
volumes:
mysql_data:
romm_resources:
romm_redis_data:
services:
romm:
image: rommapp/romm:latest
container_name: romm
restart: unless-stopped
environment:
- DB_HOST=romm-db
- DB_NAME=romm # Should match MYSQL_DATABASE in mariadb
- DB_USER=romm-user # Should match MYSQL_USER in mariadb
- DB_PASSWD=***
- ROMM_AUTH_SECRET_KEY=*** # Generate a key with `openssl rand -hex 32`
- IGDB_CLIENT_ID=*** # Generate an ID and SECRET in IGDB app:igdb_***
- IGDB_CLIENT_SECRET=*** # https://api-docs.igdb.com/#account-creation
- MOBYGAMES_API_KEY=moby_*** # https://www.mobygames.com/info/api/
- STEAMGRIDDB_API_KEY # https://github.com/rommapp/romm/wiki/Generate-API-Keys#steamgriddb
volumes:
- romm_resources:/romm/resources # Resources fetched from IGDB (covers, screenshots, etc.)
- romm_redis_data:/redis-data # Cached data for background tasks
- /media/INTENSO/romm/library:/romm/library # Your game library
- /media/INTENSO/romm/assets:/romm/assets # Uploaded saves, states, etc.
- /media/INTENSO/romm/config:/romm/config # Path where config.yml is stored
ports:
- 6464:8080
depends_on:
- romm-db
romm-db:
image: mariadb:latest # if you experience issues, try: linuxserver/mariadb:latest
container_name: romm-db
restart: unless-stopped
environment:
- MYSQL_ROOT_PASSWORD=***
- MYSQL_DATABASE=romm
- MYSQL_USER=romm-user
- MYSQL_PASSWORD=***
volumes:
- mysql_data:/var/lib/mysql |
It's a machine (a wyse thin client) I got cheap and which is running 24/7. So it seemed to me like a logical choice. I could have used my QNAP NAS as well… |
The funny(?) thing is: There is no cover-info for that game in the database as far as I can tell:
(BTW: I removed the wrong tags mentioned in #1170.) |
I created But still the custom cover is not saved. Also
Saving custom cover here
Looking into container:
|
I found the issue, but with different steps than what you reported. If I go to a game, click on "Edit", then click on "Delete cover" (trash icon), and after that I click on the pencil and add a new image, I get the behavior you are experiencing. The key here is that clicking on the "Delete cover" button first sets "remove_cover = true" to be sent in the request. When that parameter is received by the backend, it deletes the existing cover, and doesn't set a new one, even if it received it. Can you confirm if you get the right behavior, if you do not use "Delete cover", and just click on the pencil, add a new cover, and click "Save"? I'll upload a fix so that using the pencil button restarts the |
I never clicked "Delete cover" as I had none. When doing the experiment with matching with any game I alos didn't click delete if I remember correctly. But I will test it anyway now again. Tested and not clicking the trash icon made no difference.
|
* Initialize `removeCover` as `false` on each dialog render, so previous overrides don't affect the initial state. * Make both file upload and cover online search update `removeCover` to false. * Fix collection update to send `remove_cover` to the API. Potential fix for #1169.
Works! |
RomM version
Describe the bug
I'm trying to set a cover of a game but it's not saved.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
After saving the cover should be shown
Screenshots
Here are the logs of my action:
Desktop (please complete the following information):
Smartphone (please complete the following information):
Not used
The text was updated successfully, but these errors were encountered: