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

Feature/video info using title page #68

Open
wants to merge 2 commits into
base: beta
Choose a base branch
from

Conversation

guss77
Copy link

@guss77 guss77 commented Jan 9, 2016

Changed getVideoInfo to use the title page (www.netflix.com/title/{videoID}) as the old video info page was just redirecting to the main screen.

I also replaced the regular expression parser in list_video with an lxml XPath parser - its more sane that way and less breaky.

There are some readers there which I couldn't get right - the MPAA rating is gone, instead they have something called "maturity rating", so I used that instead; and I couldn't find where the quality rating is on the website at all.

I've tested it and more things work now, especially in the "my list" TV Shows section, but there is still work to be done: search, for example, sometimes still fails with an error parsing, and even if it doesn't - the list is empty.

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.

1 participant