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

Bring test branch up to date #4131

Merged
merged 112 commits into from
Mar 31, 2021
Merged

Bring test branch up to date #4131

merged 112 commits into from
Mar 31, 2021

Conversation

twanariens
Copy link
Collaborator

No description provided.

tidusjar and others added 26 commits March 24, 2021 20:57
… needed there and tidied up some of the icons
Ombi implicitly trusts that the quality override is set correctly, and causes issues downstream.

Line 258 in [TvSender.cs](https://github.com/Ombi-app/Ombi/tree/develop/src/Ombi.Core/Senders/TvSender.cs#L258) it tries to add a series but the, `AddSeries` function in [SonarrApi.cs](https://github.com/Ombi-app/Ombi/blob/develop/src/Ombi.Api.Sonarr/SonarrApi.cs#L92) fails to validate the `NewSeries` object since the `qualityProfileId` attribute is set to `0`. In the end it assumes the series is added in Sonarr and causes a API request with series id set to `0` which causes the following error.

```
[Error] StatusCode: BadRequest, Reason: Bad Request, RequestUri: http://SONARR_HOST/sonarr/api/series/0
```
rootfolder override has the same issue
Verify Sonarr quality/rootfolder override
fixes rootfolder selection in sonarr
…ble on the Discover and Search pages, added more automation tests to cover this
Fixed the v1 API, added tests around that API to ensure we keep backw…
Removed some duplicated text.
Fix minor grammatical error
kitzin - Fixed anime being send to sonarr as standard series
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants