TechnionYP5779 / team1

Git repo of team 1 of the yearly project technion CS
6 stars 3 forks source link

Issue/202/handle special days #209

Closed ofiralexi closed 5 years ago

ofiralexi commented 5 years ago

added webhook handle to phrases like - today tomorrow and now.

yoavz1997 commented 5 years ago

LGTM

yoavz1997 commented 5 years ago

I did a little more digging, are you sure you should add 2 hours to DateTime.Now?

ofiralexi commented 5 years ago

@yoavz1997 Yes, because of the time of the "local" db , that is behind on 2 hours. (You can validate it by simply printing Datetime.Now.ToString())

For example: image And as you can see, our local time now is 23:50 ( ye I ran it a minute ago...)

yoavz1997 commented 5 years ago

Ok great, thanks for clearing it up