super-cooper / memebot

A custom Discord bot For The Boys
GNU General Public License v3.0
8 stars 0 forks source link

remind me command #45

Open pfrissora opened 3 years ago

pfrissora commented 3 years ago

Is your feature request related to a problem? Please describe. nick is telling me to remind hima bout something and i can't do that too much responsiblity so have the robot do it

Describe the solution you'd like !remind (username) (amount of time minutes)

so !Remind nick#4203 120

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Add any other context or screenshots about the feature request here.

farinaanthony96 commented 3 years ago

I think the format for the command should be !remind (username) (minutes) (remind message). Otherwise, how would the command know what to remind the user of? Example: !remind nick#4203 120 “Don’t forgot to get eggs from the grocery store, Honey.“

super-cooper commented 2 years ago

Given that Discord has events now, this issue seems unimportant. However, while discussing #50, @nhawke and I considered maybe reworking this idea to function as reminders for existing events. Events do not send a push when they're starting, so the command could work like

!remind https://discord.gg/asdf?event=kjasdlkjhasdl 30m

And then the bot will tag all users marked as "interested" in the event 30 minutes before it happens. There are a couple ways this could work. Worthy of discussion.

The biggest issue with this command is datetime parsing. Do we want times to be relative to before the event? Or relative to now? What types of formats would we accept? It's likely we'd need to pull in a dependency, like dateparser, which might take on some of the complexity in determining exactly what the user wants ("30 minutes from now" and "30 minutes before" are less ambiguous).

Another question worth asking: Do we want this at all? Do we have faith that Discord will implement this natively?