sbussetti / deluge-siphon

A Chrome extension for sending torrents to your Deluge server.
http://sbussetti.github.io/deluge-siphon/
82 stars 22 forks source link

No Connection Info/Not Saving #46

Closed Cimplex closed 7 years ago

Cimplex commented 7 years ago

Hey, I've used Siphon before, but unlucky me, I stopped using it for a while and just tried it again today.

Looks like the Options page is bugging out. I can see the events on the checkboxes are working (aka, its calling "save_options()", reset is calling "clear_options()"). Going deeper, I could not actually figure out why the options are not saving.

The text boxes under "Connection Info" are not being populated.

Someone on the Chrome Extension comments edited his comment about an hour ago saying he was getting a weird error that you got resolved before.

Let me know if there is any way I can give you more information about the issue! Sorry if you are already aware/working on this. Thanks.

softgripper commented 7 years ago

I'm getting this too.

I keep getting a notification to change delugesiphon on the options page, but it's not saving any info, and it's not grabbing any of my magnetic links unless I right click them.

This has only been occurring since the last update. 0.71.1

sbussetti commented 7 years ago

Roger that. On it.

~ Steve

From: softgripper notifications@github.com notifications@github.com Reply: sbussetti/deluge-siphon reply@reply.github.com reply@reply.github.com Date: March 13, 2017 at 5:17:32 AM To: sbussetti/deluge-siphon deluge-siphon@noreply.github.com deluge-siphon@noreply.github.com CC: Subscribed subscribed@noreply.github.com subscribed@noreply.github.com Subject: Re: [sbussetti/deluge-siphon] No Connection Info/Not Saving (#46)

I'm getting this too.

I keep getting a notification to change delugesiphon on the options page, but it's not saving any info, and it's not grabbing any of my magnetic links unless I right click them.

This has only been occurring since the last update. 0.71.1

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/sbussetti/deluge-siphon/issues/46#issuecomment-286052817, or mute the thread https://github.com/notifications/unsubscribe-auth/AA3v0erIuRER-SD5GQjkbyNJzqzz5zAmks5rlQmsgaJpZM4Ma0wJ .

Churchill92 commented 7 years ago

Tagging this so I can watch for the update. I had to disable the extension from the constant spamming of the extension. Also where did the option to add your own remote server go? I don't see an option to link to my server any longer.

sbussetti commented 7 years ago

Yeah i'll have an update a little later today. This wasn't happening on my debug setup but i totally see the problem.

~ Steve

From: Churchill92 notifications@github.com notifications@github.com Reply: sbussetti/deluge-siphon reply@reply.github.com reply@reply.github.com Date: March 13, 2017 at 9:19:46 AM To: sbussetti/deluge-siphon deluge-siphon@noreply.github.com deluge-siphon@noreply.github.com CC: Steve Bussetti steve.bussetti@gmail.com steve.bussetti@gmail.com, Comment comment@noreply.github.com comment@noreply.github.com Subject: Re: [sbussetti/deluge-siphon] No Connection Info/Not Saving (#46)

Tagging this so I can watch for the update. I had to disable the extension

from the constant spamming of the extension. Also where did the option to add your own remote server go? I don't see an option to link to my server any longer.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/sbussetti/deluge-siphon/issues/46#issuecomment-286104406, or mute the thread https://github.com/notifications/unsubscribe-auth/AA3v0bm6Y5v-5O6XA0qEm6kpiE-S8tomks5rlUJygaJpZM4Ma0wJ .

drgroot commented 7 years ago

I can confirm this, the connection information boxes do not even appear.

silaha commented 7 years ago

I am getting this and found this link on a json header change in 1.3.14 on the Deluge support forum. I suspect it could be related because other Chrome extensions are not working (Darknet).

Here is the link

sbussetti commented 7 years ago

0.71.3 should fix the error on the options page. @silaha -- thanks for the link although that issue is not related to this error.

softgripper commented 7 years ago

Well, I'll try it when chrome can update it. :\

softgripper commented 7 years ago

It's still broken.

Not saving details, and not allowing to put in server stuff.

See http://imgur.com/ccc9T4P

I've started using Remote Deluge. Works great.

Churchill92 commented 7 years ago

Same Problem as before with the same picture as @softgripper .

I have no URL link to put in a remote server. I have no Save preferences option. I have no "Web UI" option that appears at all. It's as if the ability to connect to a remote server is disabled.

sbussetti commented 7 years ago

Just pushed version 0.71.4 which should help with even more cases. Let me know if anyone is still having trouble and I'll keep digging.

softgripper commented 7 years ago

So it doesn't save if there is no info in the server section. Not a huge deal.

Glad you got it at least working.

silaha commented 7 years ago

Working for me now on 0.71.4, thank you very much

sbussetti commented 7 years ago

@softgripper yeah -- once you're on the options page, it treats the server url as a required field and won't save without it. Is there a reason why you'd wanna save settings with no server url set?

softgripper commented 7 years ago

No real reason - just I went to try to save it by putting in some check boxes, and didn't get any "required field" text when I hit save. Seems to work now.

Doesn't really make sense to have it as required - I mean, if you want to just save your check boxes... you can just put rubbish in the URL. When my deluge server changes IP, or is down for maintenance, I like clearing the stuff out.

Sure, the app won't work without a server, but it wont work with an invalid one either and this is allowed :). Good old "asdfasdf" server.

Thanks for fixing the plugin. It's good now.

sbussetti commented 7 years ago

@softgripper thanks for feedback. there's room for improvement for the validation on the options page. I'll keep it in mind while I'm finishing support for "multiple servers" re: #23

Cimplex commented 7 years ago

Nice! Working for me now. Thanks!