Option to show all day events
rosenpin opened this issue · comments
Currently, we filter out all day events in the get_closest function
This is done because otherwise, the all day event will always be shown when it's taking place even if other events are more relevant
Option 1:
Make this optional using a flag
Option 2:
Show other event unless the end time of the all day event is before the start time of the closest event
I personally prefer option 2 as it will do things the 'smart' way, only show all day events if there's nothing else more interesting
What about multiple all day events?
agh, good question
maybe just print out "multiple all day events" or something like this, prompting the user to click it and view the full calendar
or maybe, print them comma separated, with max length of like 20 chars followed by ...
Example: Buy milk, Call mom, Check on...
A better way to put it would be:
If next event is tomorrow and closest all day event is today - show all day event
If more than one all day event, concat them up to N string length, order by the way Google's API provides the events
All day event 1, All day event 2, All...