Add counting clicks - Githubissues

mozilla / newnewtab

New Tab page: content server and recommendation service

https://wiki.mozilla.org/Apps/newnewtab

6 stars 5 forks source link

Add counting clicks #24

Closed zalun closed 12 years ago

zalun commented 12 years ago

Refactoring

renamed client to dbClient in app.js
added marketplaceAPIDomain to default config

tofumatt commented 12 years ago

Good start but it's a little unclear... maybe some more comments would clear things up for me. I don't get what the difference between the two click URLs are.

Sorry for some comments about code a bit unrelated to the pull; if you want to address those a bit later that's cool.

zalun commented 12 years ago

rebased as merge wasn't possible frm the pull request

tofumatt commented 12 years ago

Looks much better. Can you see about the JSON thing and after that we're good to merge it in, then I'll set it up to use the redirect system until we have the add-on to post it via AJAX?