[PR #12345] [MERGED] Add media segments API #13156

Closed
opened 2026-02-07 07:08:08 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/12345
Author: @JPVenson
Created: 7/27/2024
Status: Merged
Merged: 8/5/2024
Merged by: @nielsvanvelzen

Base: masterHead: feature/MediaSegements


📝 Commits (10+)

  • 20feb28 Added Media segment manager
  • b67498e Added "HasSegments" to MediaSourceInfo when requesting though baseitem
  • 7cd886c Fixed ordering of Media Segements
  • 6fe53ad Added media segment API controller
  • 852389d Added .ConfigureAwait(false) on media segments manager
  • 322a324 renamed MediaSegmentsController
  • 9743975 Added Model layer for Media Segments
  • a41f782 Updated media segment naming
  • 5a3829e Added mediaSegment type filter
  • 61660da Fixed codesmell

📊 Changes

13 files changed (+1124 additions, -3 deletions)

View changed files

📝 Emby.Server.Implementations/ApplicationHost.cs (+1 -0)
Jellyfin.Api/Controllers/MediaSegmentsController.cs (+61 -0)
Jellyfin.Data/Entities/MediaSegment.cs (+42 -0)
Jellyfin.Data/Enums/MediaSegmentType.cs (+39 -0)
📝 Jellyfin.Server.Implementations/JellyfinDbContext.cs (+5 -0)
Jellyfin.Server.Implementations/MediaSegments/MediaSegmentManager.cs (+106 -0)
Jellyfin.Server.Implementations/Migrations/20240729140605_AddMediaSegments.Designer.cs (+708 -0)
Jellyfin.Server.Implementations/Migrations/20240729140605_AddMediaSegments.cs (+38 -0)
📝 Jellyfin.Server.Implementations/Migrations/JellyfinDbModelSnapshot.cs (+28 -2)
📝 MediaBrowser.Controller/Entities/BaseItem.cs (+6 -1)
MediaBrowser.Controller/MediaSegements/IMediaSegmentManager.cs (+53 -0)
📝 MediaBrowser.Model/Dto/MediaSourceInfo.cs (+2 -0)
MediaBrowser.Model/MediaSegments/MediaSegmentDto.cs (+35 -0)

📄 Description

Implements media segments api

Issues
https://github.com/jellyfin/jellyfin-meta/discussions/30#discussioncomment-9629257

supersedes https://github.com/jellyfin/jellyfin/pull/10530

Please Squash


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/12345 **Author:** [@JPVenson](https://github.com/JPVenson) **Created:** 7/27/2024 **Status:** ✅ Merged **Merged:** 8/5/2024 **Merged by:** [@nielsvanvelzen](https://github.com/nielsvanvelzen) **Base:** `master` ← **Head:** `feature/MediaSegements` --- ### 📝 Commits (10+) - [`20feb28`](https://github.com/jellyfin/jellyfin/commit/20feb28d3d428a0956e71a222b65f0b37d83373a) Added Media segment manager - [`b67498e`](https://github.com/jellyfin/jellyfin/commit/b67498eb9e88c85f643c19d7951f51c82761c2ec) Added "HasSegments" to MediaSourceInfo when requesting though baseitem - [`7cd886c`](https://github.com/jellyfin/jellyfin/commit/7cd886c2a7d1237a17d3597848d2684743491603) Fixed ordering of Media Segements - [`6fe53ad`](https://github.com/jellyfin/jellyfin/commit/6fe53adb93d5c6af2260c26795d4caf84b5ebddb) Added media segment API controller - [`852389d`](https://github.com/jellyfin/jellyfin/commit/852389df36780cfe09acb26f7a0f0e9c099a6d05) Added .ConfigureAwait(false) on media segments manager - [`322a324`](https://github.com/jellyfin/jellyfin/commit/322a324cfcf2e44187749c2f929d652989957e12) renamed MediaSegmentsController - [`9743975`](https://github.com/jellyfin/jellyfin/commit/974397547d387c4598ae703f279e7342fb1dc3d3) Added Model layer for Media Segments - [`a41f782`](https://github.com/jellyfin/jellyfin/commit/a41f782237a79980b75eadb32a980d112666e9c9) Updated media segment naming - [`5a3829e`](https://github.com/jellyfin/jellyfin/commit/5a3829e9976f785d03f18c75de1b887a2999c1b0) Added mediaSegment type filter - [`61660da`](https://github.com/jellyfin/jellyfin/commit/61660da81400778c555825a08bd05ce3bbebca6b) Fixed codesmell ### 📊 Changes **13 files changed** (+1124 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Server.Implementations/ApplicationHost.cs` (+1 -0) ➕ `Jellyfin.Api/Controllers/MediaSegmentsController.cs` (+61 -0) ➕ `Jellyfin.Data/Entities/MediaSegment.cs` (+42 -0) ➕ `Jellyfin.Data/Enums/MediaSegmentType.cs` (+39 -0) 📝 `Jellyfin.Server.Implementations/JellyfinDbContext.cs` (+5 -0) ➕ `Jellyfin.Server.Implementations/MediaSegments/MediaSegmentManager.cs` (+106 -0) ➕ `Jellyfin.Server.Implementations/Migrations/20240729140605_AddMediaSegments.Designer.cs` (+708 -0) ➕ `Jellyfin.Server.Implementations/Migrations/20240729140605_AddMediaSegments.cs` (+38 -0) 📝 `Jellyfin.Server.Implementations/Migrations/JellyfinDbModelSnapshot.cs` (+28 -2) 📝 `MediaBrowser.Controller/Entities/BaseItem.cs` (+6 -1) ➕ `MediaBrowser.Controller/MediaSegements/IMediaSegmentManager.cs` (+53 -0) 📝 `MediaBrowser.Model/Dto/MediaSourceInfo.cs` (+2 -0) ➕ `MediaBrowser.Model/MediaSegments/MediaSegmentDto.cs` (+35 -0) </details> ### 📄 Description Implements media segments api **Issues** https://github.com/jellyfin/jellyfin-meta/discussions/30#discussioncomment-9629257 supersedes https://github.com/jellyfin/jellyfin/pull/10530 Please Squash --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-07 07:08:08 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#13156