CloudBot icon indicating copy to clipboard operation
CloudBot copied to clipboard

added support for optional [date] parameter for MFP plugin

Open edillingham opened this issue 7 years ago • 4 comments

uses python-dateutil to parse date strings instead of requiring a strict format

edillingham avatar Oct 02 '17 18:10 edillingham

For this one instead of adding a new requirement could you see if you could use the timeformat.py and timeparse.py in cloudbot/util/?

It does a nice job of formating time strings. Check remind.py for examples.

edwardslabs avatar Oct 04 '17 21:10 edwardslabs

Whoops didn't mean to close this.

edwardslabs avatar Oct 04 '17 21:10 edwardslabs

It doesn't look like cloudbot.util.time_parse supports anything to do with dates, which is what the plugin requires. I can remove the requirement and add a custom date parser if that's what the project prefers, but should we really reinvent the wheel?

edillingham avatar Oct 04 '17 21:10 edillingham

All PRs for gonzobot should be submitted to snoonetIRC/CloudBot now.

linuxdaemon avatar Mar 30 '18 05:03 linuxdaemon