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

Expand the remind update command to optionally edit time or user #139

Closed
kb0rg opened this issue Oct 3, 2019 · 1 comment · Fixed by #231
Closed

Expand the remind update command to optionally edit time or user #139

kb0rg opened this issue Oct 3, 2019 · 1 comment · Fixed by #231

Comments

@kb0rg
Copy link
Contributor

kb0rg commented Oct 3, 2019

Currently, \remind update <user input here> only updates the job's message with the user input.

We should expand the command to allow the user to update the job's time or at-mention as well. The revised syntax could look something like: \remind update [who|when|what] <user input here>

This issue depends on completing #138 first.

@Shadowfiend
Copy link
Contributor

This will be handled by #231 .

Shadowfiend added a commit that referenced this issue Jan 16, 2023
The new parser is custom-built and relies on extracting written patterns
from the text instead of on cron patterns. The scheduler consumes those
patterns and handles scheduling job execution. The scheduler component
maintains a sorted list of next job executions, and schedules the next
run based on the first item in that list.

An initial list of parsing and scheduler tests is included.

Still missing: formatting tests! May hold on those for now.

Fixes #139, fixes #45.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants