-
-
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
feat: add extractor for ebay video listings #6170
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does a product only ever have a single video? If so, isn't it better to use the webpage's title instead of video_json.title
?
I wasn't able to find any where there was more than one video, so I suppose it's safe to assume that they all only have a single video. I made the change to use the html title instead 👍 |
yt_dlp/extractor/ebay.py
Outdated
|
||
return { | ||
'id': video_id, | ||
'title': self._html_extract_title(webpage), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'title': self._html_extract_title(webpage), | |
'title': remove_end(self._html_extract_title(webpage), ' | eBay'), |
yt_dlp/extractor/ebay.py
Outdated
elif key == 'DASH': | ||
formats.extend(self._extract_mpd_formats(url, video_id, fatal=False)) | ||
else: | ||
self.report_warning('unsupported format found: %s' % key, video_id) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
self.report_warning('unsupported format found: %s' % key, video_id) | |
self.report_warning(f'Unsupported format {key}', video_id) |
Closes yt-dlp#6134 Authored by: JChris246
Description of your pull request and other information
This PR seeks to add an extractor for ebay video listings as described in issue #6134
Fixes #6134
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 one of the following options:
What is the purpose of your pull request?