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

[ie/jiocinema:series] Fix extraction #10139

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

varunchopra
Copy link

@varunchopra varunchopra commented Jun 8, 2024

Description of your pull request and other information

Currently, the extractor does not correctly extract all seasons of a given TV series. The API being used right now returns up to 10 seasons.

Examples:

https://www.jiocinema.com/tv-shows/mtv-splitsvilla-x5/3499820
https://www.jiocinema.com/tv-shows/mtv-roadies/3498977

You can test by running yt-dlp on the above shows with a filter like --match-filter "season_number=14" and see if the match works.

Template

Before submitting a pull request make sure you have:

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

  • I am the original author of this code and I am willing to release it under Unlicense
  • I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

@seproDev seproDev added the site-bug Issue with a specific website label Jun 8, 2024
@bashonly
Copy link
Member

Please add a test for a >10 season series URL

@bashonly bashonly changed the title Fix jiocinema extractor [jiocinema:series] Fix extraction Jun 10, 2024
@bashonly bashonly changed the title [jiocinema:series] Fix extraction [ie/jiocinema:series] Fix extraction Jun 10, 2024
@ETapk

This comment was marked as off-topic.

@ETapk

This comment was marked as off-topic.

@bashonly
Copy link
Member

@ETapk these problems are not relevant to this PR. Please open a new issue with proper log(s)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
site-bug Issue with a specific website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants