search

sierag / themoviedb-webfrontend

Awesome way to manage your rated movies, search, watchlist and more from themoviedb.org
http://www.sierag.nl/movie
GNU General Public License v3.0
7 stars 5 forks source link

Import script/procedure is horrible #21

Open blasty opened 11 years ago

blasty commented 11 years ago

Hi,

User experience and code of import.php is horrible. I fixed one "issue" with hardcoded callback URL to sierag.nl (see https://github.com/blasty/themoviedb-webfrontend/commit/bdf82ee422148e20f063349831402e1418ef6624 and https://github.com/blasty/themoviedb-webfrontend/commit/d7189b7a79b36bc7dbb30d014c43dd0e4c31365c) but more will need to be done to make this process smooth and the code nice.

I might look into this when I get bored again (expect another pullrequest!!)

sierag commented 11 years ago

I know right! Made it really easy by the minimum of code I could think of… Actually it's so ugly that the last time I saw something that looked like that ugly, I pinned a tail on it.

Regards Winnie the Pooh

On Tuesday, January 22, 2013 at 10:27 PM, blasty wrote:

Hi, User experience and code of import.php is horrible. I fixed one "issue" with hardcoded callback URL to sierag.nl (http://sierag.nl) (see blasty@bdf82ee (https://github.com/blasty/themoviedb-webfrontend/commit/bdf82ee422148e20f063349831402e1418ef6624) and blasty@d7189b7 (https://github.com/blasty/themoviedb-webfrontend/commit/d7189b7a79b36bc7dbb30d014c43dd0e4c31365c)) but more will need to be done to make this process smooth and the code nice.
I might look into this when I get bored again (expect another pullrequest!!)

— Reply to this email directly or view it on GitHub (https://github.com/sierag/themoviedb-webfrontend/issues/21).

sierag commented 11 years ago

importing single movies now enabled, you have to login to see, check single movie page. For trailers you need to add a new table to the db, check db.sql