search

SpaceK33z / web-to-plex

🚠 Adds a button on various movie & TV show sites to open it in Plex.
Other
122 stars 18 forks source link

Issue: Settings won't save #127

Closed ButtaJones closed 4 years ago

ButtaJones commented 4 years ago

Use this if Web to Plex doesn't run on any page, and you have Developer Mode enabled. Please use the table below to point to the error location.

Describe the error Settings refuse to save. Setup page keeps popping up.

To Reproduce Steps to reproduce the behavior: Tried logging in with token.

Estimated location See screenshot.

Screenshots https://i.imgur.com/cdzeuhM.jpg

Extension Information

OmNomNomagon commented 4 years ago

same issue. can't get any settings to stick using 4.1.0.22 on chrome. Also happens on the chrome store version. I have an older installation which still remembers settings fine. Only appears to occur on a fresh installation and configuration.

Ephellon commented 4 years ago

Ah, I see the problem. Also, disable the Proxy setting.

Ephellon commented 4 years ago

Fixed. Will push soon.

throw Notification... must be throw new Notification...

Ephellon commented 4 years ago

GitHub uses keywords, closed automatically.

Ephellon commented 4 years ago

Please try again @ButtaJones. Let me know if you have any issues.

ButtaJones commented 4 years ago

@Ephellon I tried. This is what happened with I tried with token. https://i.imgur.com/voeuyeW.jpg https://i.imgur.com/aGGlnjL.jpg https://i.imgur.com/sGYvZjm.png

With login/pass: https://i.imgur.com/di3NGJT.png

Ephellon commented 4 years ago

Working on it.

Ephellon commented 4 years ago

Could you tell me what you filled out please (i.e. "Just the token")

ButtaJones commented 4 years ago

@Ephellon copied my Plex token from the xml file.

eg. http://X.X.X.X:32400/library/metadata/57538?checkFiles=1&includeExtras=1&includeBandwidths=1&X-Plex-Token=[MYTOKENHERE]

I also tried with my Plex username and password. Regular username, not thru Google. Which gave me the 2nd error about the cookies.

Ephellon commented 4 years ago

I recreated the error, thank you! Found the problem, should be if(UseProxy && ProxyURL && ...), but was only if(...)

Ephellon commented 4 years ago

Pushing changes, should take 10mins

Ephellon commented 4 years ago

All done

ButtaJones commented 4 years ago

@Ephellon works. You are the G.O.A.T. my friend.

Ephellon commented 4 years ago

LOL, thanks. Let me know if anything else is broken

ButtaJones commented 4 years ago

@Ephellon I found something that should be tweaked/fixed. Open another issue?

Ephellon commented 4 years ago

Yes @ButtaJones

Ephellon commented 4 years ago

Hey @ButtaJones, haven't noticed any new issues from you. Is there still an issue you'd like to report?