Skip to content

TImezone by Abbreviation - Schedule Command #195

Open

Description

@benrick

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.

Activity

  1. Nufflee commented on Jun 7, 2018

    @Nufflee

    I'll try to do this!

  2. asyncawake commented on Jun 10, 2018

    @asyncawake
    Contributor

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

  3. Nufflee commented on Jun 11, 2018

    @Nufflee

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

  4. asyncawake commented on Jun 18, 2018

    @asyncawake
    Contributor

    @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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or requesthelp wantedExtra attention is needed

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions