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

Adds sonarr custom formats and fixes quality defs. #57

Merged
merged 35 commits into from
Sep 6, 2022
Merged

Adds sonarr custom formats and fixes quality defs. #57

merged 35 commits into from
Sep 6, 2022

Conversation

davidnewhall
Copy link
Contributor

@davidnewhall davidnewhall commented Sep 5, 2022 •
edited
Loading

  • Adds Quality Definition endpoints to Radarr.
  • Fixes Quality Definition endpoints for Sonarr.
  • Adds Custom Format endpoints to Radarr.
  • Adds new struct members to Sonarr Quality Profiles.
  • Attempts to check the response message when a non-200 is returned.

This has been reasonably tested, but more testing is ongoing.

@davidnewhall
add missing delete methods, and update a few wrrors
9cf530a
@davidnewhall
fix lint issues
4ee7ad4
@davidnewhall
Merge branch 'master' of github.com:golift/starr
a4d8bf0
@davidnewhall
Change interface, change debug logging
ddf8c65
@davidnewhall
fix what I broke
a5a408e
@davidnewhall
add back the test
5fb473c
@davidnewhall
re-export methods, and update deps
7e3e927
@davidnewhall
fix linter
8625234
@davidnewhall
remove timeout and ssl from config, use Client()
2230da3
@davidnewhall
A few fixes/updates
b65a7bb
@davidnewhall
quality defs and custom formats: radarr and sonarr
dbc17c6
@davidnewhall
Add more comments
3e41c55
@davidnewhall
bug fix
f6b0ae1
@davidnewhall
Merge branch 'dn2_change_debug' into dn2_more_endpoints
dda37dd
@davidnewhall
make Req method work with non-api paths
9bcb442
@davidnewhall
Merge branch 'dn2_change_debug' into dn2_more_endpoints
3d430b5
@davidnewhall
allow non-api paths
63f0b78
@davidnewhall
Merge branch 'dn2_change_debug' into dn2_more_endpoints
5cc32aa
@davidnewhall
add missing bodies
0479d91
@davidnewhall
Merge branch 'dn2_change_debug' into dn2_more_endpoints
972eb0f
@davidnewhall
restrict debug output to json payloads
ae96ec4
@davidnewhall
Merge branch 'dn2_change_debug' into dn2_more_endpoints
b3d1a4d
@davidnewhall
fix qualities
02e1b03
@davidnewhall
use bas quality for quality defs
492ee32
@davidnewhall
update all defs at once
9eb56a8
@davidnewhall
Add non-200 error data
5242608
@davidnewhall
ignore output on delete requests
879511c
@davidnewhall
Merge branch 'dn2_change_debug' into dn2_more_endpoints
0d5c341
@davidnewhall
merge
1bde41d
@davidnewhall
merge
b6c26a1
@davidnewhall davidnewhall mentioned this pull request Sep 5, 2022
Merged
Fuochi
Fuochi reviewed Sep 5, 2022
View reviewed changes
Copy link
Contributor

@Fuochi Fuochi left a comment

Choose a reason for hiding this comment

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

just a couple of notes

sonarr/qualitydefinition.go Show resolved Hide resolved
sonarr/qualitydefinition.go Outdated Show resolved Hide resolved
@davidnewhall
Copy link
Contributor Author

Thanks for the review! I still have some folks testing, and will merge once I'm confident this is good.

@davidnewhall
Copy link
Contributor Author

Thoroughly tested and working!

@davidnewhall davidnewhall merged commit 8e73dd5 into master Sep 6, 2022
@davidnewhall davidnewhall deleted the dn2_more_endpoints branch September 6, 2022 04:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Fuochi Fuochi Fuochi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@davidnewhall @Fuochi