RemindMeBot icon indicating copy to clipboard operation
RemindMeBot copied to clipboard

Feature request: accept ISO dates

Open chrisjbillington opened this issue 4 years ago • 5 comments

RemindMeBot doesn't seem to accept ISO dates, with requests like "RemindMe! 2022-01-01" getting the "defaulted to one day" response.

It would be good if RemindMe bot did accept ISO dates. I imagine these are simpler to parse than many of the date formats it does accept!

chrisjbillington avatar May 18 '21 01:05 chrisjbillington

That seems to work fine for me. Where are you seeing it not work?

Watchful1 avatar May 18 '21 01:05 Watchful1

Thanks for the response. Here's an example:

https://www.reddit.com/r/CoronavirusDownunder/comments/nak6ia/delayed_reopening_will_not_serve_national_goals/gyio7u4/?context=3

chrisjbillington avatar May 18 '21 02:05 chrisjbillington

Ah but this one worked fine:

https://www.reddit.com/r/RemindMeBot/comments/neeor5/remindme_bot_not_seeing_my_comments_lately/gyifwoo/?utm_source=reddit&utm_medium=web2x&context=3

chrisjbillington avatar May 18 '21 02:05 chrisjbillington

Ah, that's a common bug I've never been able to completely get rid of. The bot requires the exclamation point to trigger, but then it uses the first instance of "remindme" in the comment to get the date. So it triggered based on the command in "RemindMe! 2022-01-01", but then it tried to find the date in "Extra prompt for remindme bot, which seemed to miss this comment:" since that also has "remindme" in it. And of course there's no date in that line, so it defaulted.

I really need to rework the whole regex system that extracts the date, but there's always some edge case I break and end up giving up.

Watchful1 avatar May 18 '21 03:05 Watchful1

I see! Well, I'll keep using isodates then, and refrain from mentioning remindmebot's name in other parts of the comment to avoid triggering the existing bug.

Feel free to close this issue

chrisjbillington avatar May 22 '21 03:05 chrisjbillington