Skip to content

updated the nyaa plugin to use the new url.#1887

Merged
liiight merged 4 commits intoFlexget:developfrom
razage:update-nyaa-plugin
Jun 27, 2017
Merged

updated the nyaa plugin to use the new url.#1887
liiight merged 4 commits intoFlexget:developfrom
razage:update-nyaa-plugin

Conversation

@razage
Copy link
Copy Markdown
Contributor

@razage razage commented Jun 26, 2017

nyaatorrents.se voluntarily took itself offline because of some rumors that the "police" were coming after them. a few alternate versions popped up shortly after, and this seems to be the de facto one. I simply took the old nyaa plugin and updated the urls and values wherever necessary.

Motivation for changes:

The old nyaa plugin was using a url that no longer is online.

Detailed changes:

  • I changed the category dictionary's values because they use different numbers.
  • Their url is completely different, so I had to move around where the values get plugged in.
  • I added a ".torrent" append to the url_rewrite function because the new site has a different url structure.

nyaatorrents.se voluntarily took itself offline because of some rumors that the "police" were coming after them. a few alternate versions popped up shortly after, and this seems to be the defacto one. I simply took the old nyaa plugin and updated the urls and values wherever necessary.
@liiight
Copy link
Copy Markdown
Member

liiight commented Jun 26, 2017

You need to update tests as well

razage added 3 commits June 26, 2017 14:37
i'm unfamiliar with testing to this extent, so I'm trying my best to figure it out.
@razage
Copy link
Copy Markdown
Contributor Author

razage commented Jun 26, 2017

Sorry about that. Got the test updated and looks like it passed.

@liiight liiight merged commit ffb4140 into Flexget:develop Jun 27, 2017
@razage razage deleted the update-nyaa-plugin branch June 27, 2017 08:38
@paranoidi
Copy link
Copy Markdown
Member

@razage AWESOME job. I was checking couple weeks ago that this would be the best thing to do but had no time ...

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.

3 participants