search

leepeuker / movary

Self hosted web app to track and rate your watched movies
MIT License
412 stars 14 forks source link

Poster customization #400

Open leepeuker opened 1 year ago

leepeuker commented 1 year ago

Currently the posters used for movies are the ones used at tmdb at the time of the data sync.

Users should have the option to manually choose which poster to use from the ones available at tmdb.

The available posters should be loaded dynamically from tmdb each time (some frontend caching could be useful).

Store the manually selected image url in the database. Image caching should use this one (cache directly on setting if enabled?).

JVT038 commented 1 year ago

Maybe add an option for the users to manually upload their own poster? Or maybe add an option for the users to enter an URL to an image (like https://imgur.com/mymovieposter.jpg) and have Movary process that?

leepeuker commented 1 year ago

Good idea, that should be possible too.

JVT038 commented 1 year ago

Which users should have the permissions to change the posters? Only the owners / admins or also regular users?

leepeuker commented 1 year ago

Hm, good question. I was thinking that every user should be able to change a movie poster for their context. So if you see a movie poster at the dashboard for user one it can be different to what it looks like on the dashboard of user two. This way everyone can customize it how they want and share there preferences with others. A possible problem could be that it could be irritating when the same movie has different posters in different contexts :thinking: I am not sure yet what the best way is. Maybe we could start with only admins being able to customize posters (globally) in the first step and add more control over it later?

JVT038 commented 1 year ago

Maybe we could start with only admins being able to customize posters (globally) in the first step and add more control over it later?

Sounds like the way to go. BTW, is there a proper permission system in Movary? As far as I know, there isn't a 'real' permission / role system right? Maybe we should add that to the roadmap as well.

leepeuker commented 1 year ago

The only difference is between admin and non admin users. I think that should be enough for now, at least I had no need for more roles. The implementation of it could probably be improved