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

Fix ilcorsaroverde season search for sonarr #12613

Merged
merged 3 commits into from
Dec 4, 2021

Conversation

bakerboy448
Copy link
Contributor

worked with a prowlarr user to tweak the definition as the regular S01 query returns no results

Copy link
Contributor

@ilike2burnthing ilike2burnthing left a comment

Choose a reason for hiding this comment

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

ICV is an inconsistent mess, and uses a mix of Stagione 1, S01, and occasionally 1x02, so it's probably best to make this optional, à la ea6738c.

It might be worth adding another re_replace just for seasons, no episodes.

@garfield69
Copy link
Contributor

It might be worth adding another re_replace just for seasons, no episodes.

the regex being used does both season and season/episode combos ;-)

@ilike2burnthing
Copy link
Contributor

I swear that question mark wasn't there before... ;P

@ilike2burnthing
Copy link
Contributor

@bakerboy448 have a look and see if you're ok with that.

Copy link
Contributor Author

@bakerboy448 bakerboy448 left a comment

Choose a reason for hiding this comment

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

looks good to me

optional probably makes the most sense! - thanks

@ilike2burnthing ilike2burnthing merged commit da5c939 into Jackett:master Dec 4, 2021
@ilike2burnthing
Copy link
Contributor

v0.20.93

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.

None yet

3 participants