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.

Sign up for GitHub

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

Jump to bottom

Add configurable IgnoreDts flag to M3U tuner #3696

Merged
merged 1 commit into from
Aug 23, 2022
Merged

Add configurable IgnoreDts flag to M3U tuner #3696

merged 1 commit into from
Aug 23, 2022

Conversation

knackebrot
Copy link
Contributor

@knackebrot knackebrot commented Jun 12, 2022 •
edited by thornbill
Loading

Changes
Added configurable IgnoreDts flag

Depends on jellyfin/jellyfin#7903

Issues
Fixes jellyfin/jellyfin#7267

@knackebrot knackebrot mentioned this pull request Jun 12, 2022
Merged
@sonarcloud
Copy link

sonarcloud bot commented Jun 12, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@thornbill thornbill added enhancement Improve existing functionality or small fixes backend Requires work on the server to finish labels Jun 13, 2022
@thornbill thornbill added the stable backport Backport into the next stable release label Aug 5, 2022
@Bond-009
Copy link
Member

This should target master (same as on server side)

@Bond-009
Copy link
Member

Server side PR is now merged (to master)

@thornbill thornbill removed the stable backport Backport into the next stable release label Aug 22, 2022
@knackebrot knackebrot requested review from joshuaboniface and a team as code owners August 22, 2022 19:37
@jellyfin-bot jellyfin-bot added the merge conflict Conflicts prevent merging label Aug 22, 2022
@knackebrot knackebrot changed the base branch from release-10.8.z to master August 22, 2022 19:37
@jellyfin-bot jellyfin-bot removed the merge conflict Conflicts prevent merging label Aug 23, 2022
@sonarcloud
Copy link

sonarcloud bot commented Aug 23, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.4% 0.4% Duplication

@thornbill thornbill merged commit c94172e into jellyfin:master Aug 23, 2022
@hjschmidt99
Copy link

hey @joshuaboniface , looks like there is still a review pending, preventing this issue from making it into a release.
Could you please take a look?
Many Thanks
Hans-Jürgen Schmidt

@thornbill
Copy link
Member

No. This has been merged. It is a new feature though so it will not go in a release until a new feature release is made.

@hjschmidt99 hjschmidt99 mentioned this pull request Oct 2, 2023
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nielsvanvelzen nielsvanvelzen nielsvanvelzen approved these changes

@thornbill thornbill thornbill approved these changes

@joshuaboniface joshuaboniface Awaiting requested review from joshuaboniface

Assignees
No one assigned
Labels
backend Requires work on the server to finish enhancement Improve existing functionality or small fixes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Issue]: Audio missing in LiveTV with m3u tuner
6 participants
@knackebrot @Bond-009 @hjschmidt99 @thornbill @nielsvanvelzen @jellyfin-bot