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

feat(Streaming Service): Add Max #748

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

MiM-MiM
Copy link

@MiM-MiM MiM-MiM commented May 27, 2023

Added MAX to the streaming service list

  • Verified that HBO Max is unaffected, both Max and HBO Max are detected.

@MiM-MiM MiM-MiM changed the title (Streaming Service) Max feature(Streaming Service) Max May 27, 2023
@MiM-MiM MiM-MiM changed the title feature(Streaming Service) Max feat(Streaming Service): Add Max May 27, 2023
MiM-MiM and others added 7 commits December 13, 2023 12:48
Added MAX to the streaming service list
Decrease amount of false positives for youtube red

Currently any thing with "red" it gets youtube red detected as service, and upload script ignores provider if multiple are detected
Updated formatting (spacing)
Make Youtube RED detection case sensitive
Add Support for NOW web service
@@ -734,6 +734,7 @@
"ITV": "ITV",
"Knowledge Network": "KNOW",
"Lifetime": "LIFE",
"Max": "MAX",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be in "HBO Max" ?

Copy link
Contributor

@milkers69 milkers69 Dec 13, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MAX and HBO MAX are now different streaming services. Which one you get as a customer depends on region, but they have different encodes and selections. https://en.wikipedia.org/wiki/Max_(streaming_service)

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (bd66746) 98.46% compared to head (8c2d60f) 98.46%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #748   +/-   ##
========================================
  Coverage    98.46%   98.46%           
========================================
  Files           52       52           
  Lines         3454     3454           
========================================
  Hits          3401     3401           
  Misses          53       53           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants