DevChatterBot icon indicating copy to clipboard operation
DevChatterBot copied to clipboard

TImezone by Abbreviation - Schedule Command

Open benrick opened this issue 6 years ago • 4 comments

We should allow specifying timezone by abbreviation instead of just an offset on the schedule command

!schedule CEST should do the same thing as !schedule +2.

benrick avatar Jun 07 '18 16:06 benrick

I'll try to do this!

Nufflee avatar Jun 07 '18 16:06 Nufflee

As ajukraine suggested in chat yesterday, another nice addition would be to get the timezone by city, as in: !schedule

asyncawake avatar Jun 10 '18 08:06 asyncawake

@asyncawake Can you open a separate issue for that please?

Nufflee avatar Jun 11 '18 19:06 Nufflee

@Nufflee There is now a hangman test class. I created a test for you to code this abbreviation enhancement against. In theory, if you'd like to do that, jst uncomment it and run it against your implementation until it's green. https://github.com/DevChatter/devchatterbot/blob/master/src/UnitTests/Core/Commands/ScheduleCommandTests/ProcessShould.cs

asyncawake avatar Jun 18 '18 20:06 asyncawake