- Aug 30, 2012
-
-
Rene Saarsoo authored
Turn off logging of what MySQL session store is doing. Instead enable logging of HTTP requests that our express server gets.
-
Rene Saarsoo authored
Simple initial implementation.
-
Rene Saarsoo authored
Responds to auth and comments_meta requests. With /auth we always respond with false. With /comments_meta we only return comment counts.
-
Rene Saarsoo authored
Make the data structure the same as in old comments app.
-
Rene Saarsoo authored
-
Rene Saarsoo authored
-
Rene Saarsoo authored
On error the rows will be null, so do a check before calling rows[0].
-
- Aug 29, 2012
-
-
Rene Saarsoo authored
-
Rene Saarsoo authored
Finally understood why it's the convention in NodeJS. So following it now.
-
Rene Saarsoo authored
I assume this is more along the lines how things are done in NodeJS.
-
Rene Saarsoo authored
-
Rene Saarsoo authored
So the actual interfacing with node-mysql is contained within that class.
-
Rene Saarsoo authored
-
Rene Saarsoo authored
-
Rene Saarsoo authored
-
Rene Saarsoo authored
-
Rene Saarsoo authored
The :jasmine task now populates comments_test with test data and then runs the testsuite against it.
-
- Aug 28, 2012
-
-
Rene Saarsoo authored
-
Rene Saarsoo authored
The script update_votes.sql calculates the values for that column based on the entries in votes table. This is an optimization to avoid aggregating the votes in a separate table every time we retrieve comments. Also updated examples.sql to use the new full_visible_comments view - this greatly simplified all queries.
-
Rene Saarsoo authored
Include jasmine-node into our comments backend dependencies. Write comments.spec.js to test all methods in Comments class. For now we're using the actual full database - must replace with a test database. Add :jasmine task to Rakefile.
-
Rene Saarsoo authored
It also turned out that I can't have comments inside package.json. So I need to remove those helpful comments from there... great.
-
Rene Saarsoo authored
Also add full_visible_comments view to simplify the queries. Finally added a small test script to check if it's actually working.
-
- Aug 24, 2012
-
-
Rene Saarsoo authored
-
Rene Saarsoo authored
-
Rene Saarsoo authored
-
Rene Saarsoo authored
-
Rene Saarsoo authored
-
Rene Saarsoo authored
-
Rene Saarsoo authored
-
Rene Saarsoo authored
Results in better error messages.
-
Rene Saarsoo authored
A strange 'unknown' target has NULL fields otherwise.
-
Rene Saarsoo authored
-
Rene Saarsoo authored
-
Rene Saarsoo authored
MySQL has restriction for index size, so make the fields shorter. 100 should be enough for class and member names. 50 should be enough for domain name.
-
Rene Saarsoo authored
-
Rene Saarsoo authored
-
Rene Saarsoo authored
-
Rene Saarsoo authored
-
Rene Saarsoo authored
We'll delete these targets later, but for now I want to be able to insert them.
-
Rene Saarsoo authored
I though Date.parse() returns a date, but it returns milliseconds instead.
-