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

[extractor/stageplus] Add extractor #6838

Merged
merged 2 commits into from Apr 18, 2023

Conversation

bashonly
Copy link
Member

stage-plus.com is a subscription-only site that streams live/VOD concerts and albums, mostly of classical music.

This PR only adds support a VOD concerts, not live concerts or albums.

Each VOD concert is a playlist made up of multiple works (videos). I opted not to make them multi_video playlists, because each work is then broken down into multiple movements (chapters).

There is a huge graphQL request body, but most of the metadata being requested is either useful or potentially useful (if/when livestream/album extractors are added). It seemed more efficient to leave it intact than to try and carefully perform surgery on it, only to maybe have to restore parts later.

Closes #6806

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?

@bashonly bashonly added the site-request Request to support a new website label Apr 16, 2023
@pukkandan
Copy link
Member

There is a huge graphQL request body, but most of the metadata being requested is either useful or potentially useful (if/when livestream/album extractors are added). It seemed more efficient to leave it intact than to try and carefully perform surgery on it, only to maybe have to restore parts later.

Add a #TODO that it need to be cleaned up after live is impl

@bashonly bashonly merged commit e5265dc into yt-dlp:master Apr 18, 2023
11 checks passed
@bashonly bashonly deleted the extractor/stageplus branch April 18, 2023 02:28
aalsuwaidi pushed a commit to aalsuwaidi/yt-dlp that referenced this pull request Apr 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
site-request Request to support a new website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Site support request: STAGE+
2 participants