-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Add media segments API #12345
Add media segments API #12345
Conversation
Changes in OpenAPI specification found. Expand to see details.What's New
|
removed empty route
Jellyfin.Server.Implementations/MediaSegments/MediaSegementManager.cs
Outdated
Show resolved
Hide resolved
Jellyfin.Server.Implementations/MediaSegments/MediaSegementManager.cs
Outdated
Show resolved
Hide resolved
Fixed review comments Media segments
refactored api and manager usage
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.
Approved from an API perspective. Few typos to fix.
Jellyfin.Server.Implementations/MediaSegments/MediaSegmentManager.cs
Outdated
Show resolved
Hide resolved
Made mediasegment filter optional
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.
SegmentProvider interface for plugins and its integration is missing
Not part of this PR. Will follow later. |
Jellyfin.Server.Implementations/MediaSegments/MediaSegmentManager.cs
Outdated
Show resolved
Hide resolved
|
part 2 is in #12359 |
|
Jellyfin will no longer start. ApplicationHost needs |
Implements media segments api
Issues
jellyfin/jellyfin-meta#30 (comment)
supersedes #10530
Please Squash