Radio cover image

[TrackArcher]

Navidrome since recently supports radio station cover art in the api. Instead of using the workaround of HomePageUrl to map album art it now can be mapped directly. If I understand what it does it should work for the mobile app and android auto. but I am degoogled so I can't verify if it works for android auto. The pictures can be changed through the browser but maybe a future pull request can add it to be done directly in the app.

[TrackArcher]

this solves #716

[eddyizm]

Hi @TrackArcher
Today's merges caused some conflicts with your PR.
If can update it, I will review this next and try and get it in a new pre release soon.

[MaFo-28]

Hello,
There are files that should not be modified: AppDatabase.java, CHANGELOG.md‎
On Android Auto, artworks are already correctly displayed; you need to check that there is no regression with your modifications.

[TrackArcher]

Hello, There are files that should not be modified: AppDatabase.java, CHANGELOG.md‎ On Android Auto, artworks are already correctly displayed; you need to check that there is no regression with your modifications.

Thanks, I will look into it

[TrackArcher]

I didn't change CHANGELOG.md as far as I know and AppDatabase.java is now as in the development branch. there are no merge conflicts now. As far as android auto, I believe it uses the same logic as the mobile app for the radio artwork. However as I said, I have no ways of checking since I don't have android auto. Thanks.

edit: After merging development, my earlier PR build already added cover_art at v18. The merged auto-migration 18→19 tries to add it again → duplicate column crash. Needs a v20 migration (or maintainer-handled migration). I haven't modified AppDatabase.java per review feedback.

[eddyizm]

Hi @TrackArcher ,
I think you need to clean your local dev environment - I've had this problem before when multiple PR's are doing database changes. I am not sure where you had started your developement but right now dev is on 19, as you noticed so AppDatabase.java will need to have a migration and bumped to 20 in your PR so since you are changing the model by adding columns
Once we get that, we will need to test the Android auto issue cause that is harder for me but you can test it with the DHU to emulate it on (since i don't have AA in my car either)

[TrackArcher]

Yes, I will have time to look into it later this week. I can't verify android auto because I am degoogled and can't use the android auto app, so I'd need someone else to test android auto.