chr-1x / hmh_bot

Python modules for the Willie IRC bot used in the Handmade Hero twitch chat.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Specify timezones for !schedule command (default output)

chr-1x opened this issue · comments

People are lazy

This was fixed by itsuart's pull request, but is still in a separate command. Should we attempt to do more complex argument parsing to merge it into a single command?

I tried to "patch" !schedule command first, but found that would require changing a lot of not directly related stuff, so resorted to a new command.
I wonder if we should allow people to record their tz (like !set-tz CET) and then all absolute time (ones with date and time) related commands will look for that record and if it exist - use that timezone, if not - use default (PST). I don't think there is much demand for specifying tz in commands. That would also get rid of !tzschedule command and having less code is always better.

Certainly possible -- the bot framework pretty easily allows storing information on a per-nickname basis. It's something we should look into as development continues.