-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Fix LiveTV Playback #6166
Fix LiveTV Playback #6166
Conversation
4e2a3de
to
95e2387
Compare
@dmitrylyzo Also is mediaSourceId not working with live TV a server bug? (returns NoCompatibleStream). Would it be better to fix this on the server than workaround it here? |
Probably. Here and here Possible fix: we can make them mutually exclusive: jellyfin-web/src/components/playback/playbackmanager.js Lines 459 to 464 in 3a33ed9
if (liveStreamId) {
query.LiveStreamId = liveStreamId;
} else if (mediaSourceId) {
query.MediaSourceId = mediaSourceId;
} There was a regression in the server that was recently fixed. Made a comment on that issue. But now I think that it fails here. I am surfing the code from mobile and could be wrong. |
I should note that liveStreamId isn't set by
|
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.
I tested Live TV with these changes on the 10/7/24 10.10 build, and live TV works with these changes.
I did not do additional testing with DefaultAudioStreamIndex
or DefaultSubtitleStreamIndex
.
Co-authored-by: Dmitry Lyzo <56478732+dmitrylyzo@users.noreply.github.com>
Co-authored-by: Bill Thornton <thornbill@users.noreply.github.com>
Quality Gate passedIssues Measures |
Cloudflare Pages deployment
|
Fixes LiveTV playback not working when specifying mediaSourceId
Changes
Issues
Fixes #6164