bot icon indicating copy to clipboard operation
bot copied to clipboard

Adding extra functionality and refinements to !remind

Open scragly opened this issue 6 years ago • 7 comments
trafficstars

Possible improvements have been discussed for the reminder command:

  • Repeating reminders e.g. !remind "@Admins meeting in 5 minutes" every Sunday at 20:55
  • More natural time parsing
  • DM reminders
  • Allowing reminders in functional channels such as #dev-contribs which currently is disallowed for non-staff.

scragly avatar Nov 15 '19 14:11 scragly

Adding a couple more enhancements:

  • Add reminder ID to the confirmation embed
  • Add the target datetime to the confirmation embed (in addition to the currently provided timedelta)

sco1 avatar Feb 12 '20 13:02 sco1

I seem to have forgotten to add also:

  • Jump link to original invoke msg in non-repeating reminder msg (not the confirmation msg)

scragly avatar Feb 12 '20 21:02 scragly

That was added by #466

sco1 avatar Feb 12 '20 21:02 sco1

Oh, very nice.

scragly avatar Feb 12 '20 21:02 scragly

For scoping this should be split into a few discrete chunks:

Chunk the First (low hanging fruit) #749

  • Reminder ID in confirmation embed
  • Target datetime in confirmation embed
  • Expand whitelist for non-staff

Chunk the Second (requires backend changes)

  • Repeating reminders
  • DM reminders

Chunk the Third (converter change has broader implications)

  • More natural time parsing

sco1 avatar Feb 12 '20 22:02 sco1

For the more language time parsing feature, can we make use of parsedatetime?

Shivansh-007 avatar Mar 04 '21 16:03 Shivansh-007

we already have arrow as a dependency, so maybe we can make use of that?

ChrisLovering avatar Mar 04 '21 16:03 ChrisLovering

The points here other than "Repeating reminders" and "DM reminders" seem to have been completed or have their own tracking issues. I don't think repeating/dm reminders are super important so i'll close this and let anyone create separate issues for them if they have a use case.

wookie184 avatar Apr 14 '24 17:04 wookie184