-
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
[watchertv] Add extractor #9748
base: master
Are you sure you want to change the base?
Conversation
I made the dropout extractor a little more generic and then extended it to make the watcher extractor. If this is not the correct way to do this, please point me in the right direction. |
You are on the right track. But inheriting from concrete IEs commonly cause a lot of issues and so we try not to do that. You can create a Alternatively, a single IE can be made to handle both |
I went with the Since different |
Co-authored-by: pukkandan <pukkandan.ytdlp@gmail.com>
IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information
Adds extractor for WatcherTV. Requires login credentials with active subscription. Modified dropout extractor to be a little more generic and extended it to make the watcher extractor. They are identical in their site functionality.
Fixes #
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:
What is the purpose of your pull request?