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

New in v2.0.0 - Incorrect TMDB info for people #60

Closed
bigdork opened this issue Oct 26, 2020 · 5 comments · Fixed by #66
Closed

New in v2.0.0 - Incorrect TMDB info for people #60

bigdork opened this issue Oct 26, 2020 · 5 comments · Fixed by #66

Comments

@bigdork
Copy link

bigdork commented Oct 26, 2020

Snippet from config, but it's not just Chappelle:

Actor Dave Chappelle:
    sort_title: -5 Actor Dave Chappelle
    actors: Dave Chappelle
    #tmdb_id: 4169
    tmdb_summary: 4169
    tmdb_poster: 4169
    collection_order: release

When running, the incorrect poster (for Chappelle) is loaded:
| Detail: poster updated to [url] https://image.tmdb.org/t/p/original/m2iVkJPxDrrink7NiE412nFBMQj.jpg

This is the incorrect poser, as shown.
Screen Shot 2020-10-25 at 9 09 56 PM

The URL to the proper poster (Chappelle) should be https://image.tmdb.org/t/p/original/qrJzigQLQ7FwrzYBLeDwvuPN63t.jpg

Also, not sure if it is known but tmbd_id still doesn't work :(

@bigdork
Copy link
Author

bigdork commented Oct 26, 2020

Note, this was working properly pre-v2.0.0.0.

@meisnate12
Copy link

ill update the readme but tmbd_id isnt supposed to work with actors just with collections and movies i could add a tmdb_actor attribute that could do both summary and poster as for the error give me a little bit im gonna try and figure out whats going on

@meisnate12
Copy link

so its finding the tmdb movie 4169 which is the intended use since 2.0.0 what I'll add to 2.1.0 is tmdb_actor and when you specify 4169 there it will update everythin correctly

@bigdork
Copy link
Author

bigdork commented Oct 26, 2020

so its finding the tmdb movie 4169 which is the intended use since 2.0.0 what I'll add to 2.1.0 is tmdb_actor and when you specify 4169 there it will update everythin correctly

Ack, I noted Chappell because that's what is shown (now, incorrectly) in the readme in essentially the same format as I've quoted above. Also note, not just actors - Writers and Directors are as well.

@bigdork bigdork changed the title New in v2.0.0 - Incorrect TMDB Posters New in v2.0.0 - Incorrect TMDB info for people Oct 26, 2020
@meisnate12
Copy link

its a change in tmdb_summary and tmdb_poster that allows their queries to be more accurate but it lost the weird way they were doing peoples posters. I've got this almost all ready to go in 2.1 so youll see the fix coming soon.

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