please add support for nzbs.in (NewsNab) about sabconnectplusplus HOT 8 CLOSED

That commit is to support multiple sites that are using the Newznab software to index NG. If nzbs.in uses Newznab, there is nothing to do; just add "nzbs.in" in your options, in the Newznab providers text field.
If it's not using Newznab, you're better off copying another provider. The one to use would depend on which one looks the most like nzbs.in.

from sabconnectplusplus.

Oh, just saw in the subject that it is based on Newznab. So like I said, just add it in your Sabconnect++ options, and it should work. If it doesn't, they probably modified the standard Newznab templates, so you'd need to modify our Newznab injection to support both the default and nzbs.in.

from sabconnectplusplus.

ya, tried that already; had no effect.

from sabconnectplusplus.

nzbs.in operation works fine. Some recent changes I submitted were tested mainly against nzbs.in.

I am in the process of debugging some quirks with the support though, as it occasionally grabs no category (despite the fact a refresh might).

Can you provide reproduction steps to where you are on the site and I will check it further for you?

from sabconnectplusplus.

Hm! Oddly the SAB download icon now appears fine in Chrome and clicking it correctly sends the download to my configured SABnzbd install… so nevermind! Sorry 👍

from sabconnectplusplus.

When you visit nzbs.in, ensure you allow the page to completely load before switching tabs.

There is a general bug in the newznab function that is fixed in the next release that should allow it to work regardless of which tab you have open when the extension adds the buttons.

from sabconnectplusplus.

Ah, yeah... That funny bug. Mind if I release now @ppslim ?

from sabconnectplusplus.

Go for it, was never really my decision anyway 👍

from sabconnectplusplus.