styxit / HTPC-Manager

A fully responsive interface to manage all your favorite software on your Htpc.
http://htpc.io
MIT License
529 stars 183 forks source link

Not connecting to CouchPotato #254

Closed gavsiu closed 10 years ago

gavsiu commented 10 years ago

I've tried checking for updates and it says it's up to date.

In CouchPotato settings, password field is not masked and fails when I press the test button.

2014-07-08 11:55:10 cherrypy.error.34465656976  ERROR   [08/Jul/2014:11:55:10] HTTP Traceback (most recent call last):
2014-07-08 11:55:10 modules.couchpotato ERROR   Unable to connect to couchpotato
Hellowlol commented 10 years ago

Working fine here. however the get api thing isnt working. dunno if they have changed the api again.

styxit commented 10 years ago

Maybe we should remove fetching the api key by entering username and password, and just let the user provide the api key. Like it used to be?

Hellowlol commented 10 years ago

Ill rather fix it. The user just have to write the apikey manually and press save until it's fixed.

Den 9. juli 2014 kl. 10:01 skrev Styxit notifications@github.com:

Maybe we should remove fetching the api key by entering username and password, and just let the user provide the api key. Like it used to be?

— Reply to this email directly or view it on GitHub.

styxit commented 10 years ago

I merged the PR. Everything working now?

Ps: @Hellowlol you might want to take a look at your git settings. Al your commits show up as 'john' and are not linked to your github account. Make sure your email in git settings matches your mail address used to sign up on github. See https://github.com/styxit/HTPC-Manager/commit/6aa765b82acff5958b548cf4f12dd2d722766e5a

Hellowlol commented 10 years ago

It works here.Thanks for the tip. I have updated the email, but it didnt help :/

gavsiu commented 10 years ago

Test button works and does not fail anymore.

Password field is still not masked. Maybe set the input field type to password instead of text?

Also, some of the posters are showing up as broken links. I don't know if I should create a new issue or not. It's fine in CP. I tried refreshing the movies in HTPC Manager.

Hellowlol commented 10 years ago

Good to hear it's working again. If the posters is missing its because that movies poster value is empty. I could probably check for that and try to grab some other. The password input field should be changed.

It will take some time before I have a chance to fix it. So if any is up for the task feel free.

Den 14. juli 2014 kl. 21:58 skrev gavsiu notifications@github.com:

Test button works and does not fail anymore.

Password field is still not masked. Maybe set the input field type to password instead of text?

Also, some of the posters are showing up as broken links. I don't know if I should create a new issue or not. It's fine in CP. I tried refreshing the movies in HTPC Manager.

— Reply to this email directly or view it on GitHub.

gavsiu commented 10 years ago

I assumed it automatically pulls all info including posters from CouchPotato database.

You said the posters are missing because the value is empty. I checked using Firebug and this is the html source for the image:

<img width="100" height="150" src="/couchpotato/GetImage?w=100&amp;h=150&amp;url=https://image.tmdb.org/t/p/w154/2ZFmSkIihM3cWTjtKVy1UAchG5L.jpg" class="thumbnail">

Pasting https://image.tmdb.org/t/p/w154/2ZFmSkIihM3cWTjtKVy1UAchG5L.jpg into my browser works and shows an image.

Perhaps the broken image is due to when I was setting CouchPotato up. I tried turning CouchPotato integration in HTPC Manager on and off, but it didn't do anything. Is there a way to clear the cache/database or something?

Hellowlol commented 10 years ago

I don't think it's because how you have set up cp, it's prob something to do with pil. I know no other way other then to delete the images folder.

gavsiu commented 10 years ago

Ok. Deleting the folder fixed all but 1 poster that has an undefined value.

Hellowlol commented 10 years ago

What image is it suppose to be? (What movie)

gavsiu commented 10 years ago

Mortal Instruments: City of Ashes, The. Wait. I remember now. That movie is TBA and doesn't have a poster in CP. All is well. Thanks.