mozilla / newnewtab

New Tab page: content server and recommendation service
https://wiki.mozilla.org/Apps/newnewtab
6 stars 5 forks source link

Decouple database connection, express app and setting system #22

Closed zalun closed 12 years ago

zalun commented 12 years ago

Current config logic requires using express even for command line. We should use nconf (https://github.com/flatiron/nconf) or a simple JSON config. I've been discussing it on freenode#express and they suggested using just a default.json file that gets loaded by default, and then a {DEV_ENV}.json file that is loaded on top of that to overwrite settings.

zalun commented 12 years ago

Done in https://github.com/mozilla/newnewtab/pull/23/