search

mozilla / chronicle

find everything you've ever found
http://mozillachronicle.tumblr.com/
Mozilla Public License 2.0
16 stars 6 forks source link

refactor(db): Change the test data time interval so we see more date headings #346

Closed pdehaan closed 9 years ago

pdehaan commented 9 years ago

Each successive test data visit should be +5 hours later than the previous one, giving us 5-6 entries per date heading.

FWIW, 5 [hours] is a magically random number that I made up with no significance, apart from using 2, 3, 4, 6, or 8 were all divisible by 24, giving us the exact same numbers of entries per day. Or we could increase this to an equally arbitrary 7 or 9 hours difference, if that floats your collective boats.

Fixes #345

pdehaan commented 9 years ago

r? @6a68 — I could use some help with this. It looks so obvious that it hurts, but if you clone+run this and then do the ./bin/create_db.sh > ./bin/migrate.js > ./bin/create_test_data.js dance, it seems to give me most unexpected headings. For example, I didn't know there was a January 0th, but I seem to have that heading. Not sure if I messed something up, or if something else is afoot.

chronicle

Figure 1: What the hell damn guy?!

pdehaan commented 9 years ago

Date heading bug fixed in #349

jaredhirsch commented 9 years ago

sure, let's do it