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

NZBDrone Module #375

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

NZBDrone Module #375

wants to merge 8 commits into from

Conversation

gugahoi
Copy link
Collaborator

@gugahoi gugahoi commented Aug 24, 2014

So I was working on this a couple of months back. It is still unfinished but given that someone else also made a PR for it I thought I should present you guys with the option as well. I'll leave you guys to chose the one you prefer.

@gugahoi gugahoi changed the title NZBDrone Moduel NZBDrone Module Aug 24, 2014
@dlove67
Copy link

dlove67 commented Aug 25, 2014

Not sure if it could help, but if add shows still isn't working, the code that handles that are lines 212-274 in my nzbdrone.py. In particular Lines 244 for the payload, and lines 263-265 for the POST to nzbdrone.

Not sure if you have or not, but it may be worthwhile to add a selector for anime/standard/daily as well. It's not documented in the API but you can send the options.

@gugahoi
Copy link
Collaborator Author

gugahoi commented Aug 25, 2014

Thanks dlove67! I'll definitely see your implementation when I get the chance. To be honest I broke my nzbdrone install several times trying to get the "add show" to work and I got fed up with it. It was a silly system that was in place for their API and I actually found some bugs with drone at the time. Luckily the devs were very quick to fix anything I found. I have not done anything related to anime stuff since at the time it there was no support for it but it's worth taking it into consideration.

@gugahoi
Copy link
Collaborator Author

gugahoi commented Aug 31, 2014

Also would be great to have @N3MIS15 or @mrkipling chime in here since I will obviously be biased and feel like it's important to be fair on this one. 😄

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

2 participants