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

Added provider for nzbX. #567

Merged
merged 2 commits into from
Feb 23, 2013
Merged

Conversation

jrdnull
Copy link
Contributor

@jrdnull jrdnull commented Jan 25, 2013

Provider for nzbX, discussed with thezoggy and Lemon in irc.

Let me know of any changes/improvements and I'll fix them.

This was referenced Jan 27, 2013
@thezoggy
Copy link
Contributor

This is good to go.

@midgetspy
Copy link
Owner

This code seems to work well but the provider itself is providing junk results. I won't be pulling it for now, hopefully we can figure out a way to exclude results like these:

[Desktop]-[1/1] - "europolis.2010.pdtv.x264-saftoiu.nfo.nzb"
[o4u] "Melissa & Joey.s01e01.Pilot.dTV.x264-o4u.vol07+08.PAR2"
Scary.Walls.(08-10-12).PDTV.XviD-acrwt

(and before somebody asks: no SB won't add parsing rules to parse the raw post names, that's the entire point of the indexer)

@LemonadeDev
Copy link

I've found the issue, the API is returning spam flagged results to you (as seen by the spam: 1 attribute). I'll have these hidden in the next couple of minutes.

@LemonadeDev
Copy link

Appears there was an issue with the "recent" code when using more than one category, I have now fixed this.

@thezoggy
Copy link
Contributor

ty lemon

@jrdnull
Copy link
Contributor Author

jrdnull commented Feb 21, 2013

Have you had a chance to look at this again midgetspy?

@midgetspy midgetspy merged commit 82d204b into midgetspy:development Feb 23, 2013
gborri pushed a commit to gborri/Sick-Beard that referenced this pull request Jul 16, 2014
Add new feature, check propers interval.
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.

4 participants