search

AdmiralGT / ttennis_ladder

0 stars 1 forks source link

Add date to each match #15

Open andrew-nubbert opened 7 years ago

andrew-nubbert commented 7 years ago

Include the date each game was played. Show this in places where matches are displayed (recent results on main page and on per-player pages).

This could then be used in various ways, like "biggest upsets of the last month" or "date of highest achieved rating for each player".

ghost commented 7 years ago

Nice idea, but is this going to involve a "database" update? what do we do about existing results?

andrew-nubbert commented 7 years ago

Good points. This would indeed require a database update, and some way of coping with not having a date for existing results. Could well be awkward. E.g. if you use a placeholder date ("1st October 2016") for all existing matches, it'll appear in lots of the stats. If you use "Unknown" it'll require special case code to avoid trying to parse it as a date.

So yes, maintaining back-compatibility with existing results sounds like a bit of a pain, and this may not give us the best bang-for-buck.