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

[metadata.themoviedb.org.python] v1.4.2 #308

Merged
merged 2 commits into from
Oct 9, 2021
Merged

[metadata.themoviedb.org.python] v1.4.2 #308

merged 2 commits into from
Oct 9, 2021

Conversation

mkaflowski
Copy link

Description

Feature: downloading logos from tmdb:
Priority:

  • fanart
  • tmdb from your language
  • general tmdb logo (without language)
  • tmdb from "en"

Feature: search language option added
This is useful if you have other language than "en". For example I hava "pl" and for movie "My Son 2021" I will get wrong result because the right one has polish translation.
I have added new string but is missing for many language (dont know how to add it fast).

Change: searching movies from differnt years if not found
I have file "Nic Smiesznego 1995" for example. But the movie is from 1996 and will not be found. Now it is tried to find from +/- 1 year and without year if not found.

Checklist:

  • My code follows the add-on rules and piracy stance of this project.
  • I have read the CONTRIBUTING document
  • Each add-on submission should be a single commit with using the following style: [plugin.video.foo] v1.0.0

Python Scraper update: logos, search languege, finding movies from other years
@mkaflowski
Copy link
Author

Yes. Corrected.

@razzeee razzeee self-requested a review October 9, 2021 20:49
@razzeee razzeee changed the title Python Scraper update: logos, search languege, finding movies from other years [metadata.themoviedb.org.python] v1.4.2 Oct 9, 2021
@razzeee razzeee merged commit 6d68589 into xbmc:matrix Oct 9, 2021
@razzeee
Copy link
Member

razzeee commented Oct 9, 2021

Please follow the commit message guidelines the next time. I've changed them for you this time.

Thanks

@rmrector
Copy link

WTF? This was not merged upstream. This scraper should still support Leia, but that got thrown out the window too.

@razzeee
Copy link
Member

razzeee commented Oct 15, 2021

The code should be there (I'm on mobile), but your right, that the version wasn't formally released

@rmrector
Copy link

This was not merged upstream.

I didn't make that up, half of this code wasn't upstreamed until today, after changes from a code review.

xbmc/metadata.themoviedb.org.python#101

It was noticed by others: xbmc/metadata.themoviedb.org.python#104

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

Successfully merging this pull request may close these issues.

None yet

3 participants