Skip to content

Update to BTN API - #982

Open
chinedu40 wants to merge 1 commit into
midgetspy:developmentfrom
chinedu40:patch-1
Open

Update to BTN API#982
chinedu40 wants to merge 1 commit into
midgetspy:developmentfrom
chinedu40:patch-1

Conversation

@chinedu40

Copy link
Copy Markdown

No description provided.

@thezoggy

Copy link
Copy Markdown
Contributor

you shouldnt change the line endings.. aka it shows you modified the whole file.... which makes it hard to see actual changes and will most likely delay this from ever getting pulled

@chinedu40

Copy link
Copy Markdown
Author

Apologies, I'm new to this. Is there a way to just show the changed line?

@voxxit

voxxit commented Feb 22, 2017

Copy link
Copy Markdown

@noone5

noone5 commented Mar 2, 2017

Copy link
Copy Markdown

I have the following error:

2017-03-02 14:52:30 SEARCHQUEUE-MANUAL-SEARCH :: Unknown error while accessing BTN: 'SafeTransport' object has no attribute 'context'
2017-03-02 14:52:30 SEARCHQUEUE-MANUAL-SEARCH :: No data returned from BTN

@ekohl

ekohl commented Mar 2, 2017

Copy link
Copy Markdown

@noone5 for that joshmarshall/jsonrpclib#36 is needed jsonrpclib should be updated as well. Luckily BTN is the only provider that uses it so that simplifies testing.

@TVViewer

TVViewer commented May 8, 2017

Copy link
Copy Markdown

scrap this in favour of #984 which is a more elegant fix

ekohl added a commit to ekohl/Sick-Beard that referenced this pull request Jul 1, 2017
This is midgetspy#982 but without all
the whitespace changes. It depends on bumping jsonrpclib for proper
https support.

midgetspy#984 does update the URL but
doesn't switch to HTTPS.
ekohl added a commit to ekohl/Sick-Beard that referenced this pull request Jul 1, 2017
This is midgetspy#982 but without all
the whitespace changes. It depends on bumping jsonrpclib for proper
https support.

midgetspy#984 does update the URL but
doesn't switch to HTTPS.
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.

6 participants