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(starr-german): Add Groups to German Tiers #2250

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Lokilicious
Copy link
Contributor

@Lokilicious Lokilicious commented Feb 2, 2025

Pull Request

Purpose

  • FENDT to German Scene (Sonarr)
  • ACED to German Scene (Sonarr)
  • FKKTV to German Scene (Sonarr)
  • euHD in German Scene (Sonarr)
  • OCA in German Scene
  • MAMA in Tier 01 (Radarr)
  • 4SJ in 4SF Aliases
  • RIPLEY in German Scene (Sonarr)
  • GTVG in German Scene (Sonarr)
  • JellyfinPlex in German Microsized https://www.xrel.to/p2p/group-5586-jellyfinplex/releases.html
  • Jellyfin-Plex in German LQ (release title)

Approach

Add groups to CFs

Requirements

@github-actions github-actions bot added Area: Sonarr Sonarr Related Area: Backend Backend Changes, not related to a specific section Area: Starr Custom Formats Issue is related to custom formats labels Feb 2, 2025
@Lokilicious Lokilicious changed the title fix(starr): Add Groups to German Scene fix(starr-german): Add Groups to German Tiers Feb 2, 2025
@Lokilicious Lokilicious changed the title fix(starr-german): Add Groups to German Tiers feat(starr-german): Add Groups to German Tiers Feb 2, 2025
@github-actions github-actions bot added the Area: Radarr Radarr Related label Feb 6, 2025
@Lokilicious Lokilicious marked this pull request as ready for review February 6, 2025 09:26
@Lokilicious Lokilicious requested a review from TRaSH- February 9, 2025 11:35
"negate": false,
"required": false,
"fields": {
"value": "Jellyfin-Plex$"
Copy link
Contributor

Choose a reason for hiding this comment

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

Just to be sure, doesn't "Jellyfin-Plex$" need to be "^(Jellyfin-Plex)$"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As this is a not a release group condition but a release title condition ^(Jellyfin-Plex)$ would not work.

Not 100% sure but a release group condition only matches the string after the last "-" right? So since the Release Group has a "-" in it we need a release title condition to match it properly.

Correct me if im wrong in my assumption.

"negate": false,
"required": false,
"fields": {
"value": "Jellyfin-Plex$"
Copy link
Contributor

Choose a reason for hiding this comment

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

Just to be sure, doesn't "Jellyfin-Plex$" need to be "^(Jellyfin-Plex)$"?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Backend Backend Changes, not related to a specific section Area: Radarr Radarr Related Area: Sonarr Sonarr Related Area: Starr Custom Formats Issue is related to custom formats
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants