search

mozilla / newnewtab

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

sample local settings file #19

Closed zalun closed 12 years ago

zalun commented 12 years ago

I think it's the otherwise copying settings.js over settings-local.js means that a dev might not realise original settings.js got updated. We might change settings.js to add many things. Dev might change quite a lot of things to use locally. This solution removes the need to copy settings.js every time they are changed. That's a style used in all Django projects I know.

tofumatt commented 12 years ago

Snap, I misunderstood what this was because I was at dinner and on my phone. Yes. This is awesome. Maybe add test(s)? Sorry for my earlier dumbness.

tofumatt commented 12 years ago

We usually call the file settings-local.js-dist not -diff though.

zalun commented 12 years ago

yep - typo - the diff thing as of the test ... we can't do it - settings-local doesn't exist by default and test should work as such I tested it "manually" by changing the port