Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remind me command #45

Open
pfrissora opened this issue Sep 3, 2020 · 2 comments
Open

remind me command #45

pfrissora opened this issue Sep 3, 2020 · 2 comments
Assignees
Labels
feature New feature or request good first issue Good for newcomers low-priority Low priority issues for the back burner

Comments

@pfrissora
Copy link

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.

@super-cooper super-cooper added good first issue Good for newcomers low-priority Low priority issues for the back burner feature New feature or request labels Sep 3, 2020
@farinaanthony96
Copy link
Collaborator

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
Copy link
Owner

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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request good first issue Good for newcomers low-priority Low priority issues for the back burner
Projects
Status: In progress
Development

No branches or pull requests

4 participants