Skip to content

feat: /recurrent command to add recurring messages #45

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

Closed
wants to merge 10 commits into from

Conversation

luca-montaigut
Copy link
Contributor

@luca-montaigut luca-montaigut commented Aug 31, 2023

Manage recurring messages with con job:
/recurrent +

  • add : recurring message daily / weekly / monthly
  • remove : recurring message
  • list : recurring message

@luca-montaigut luca-montaigut force-pushed the add-recurring-message-feature branch from 490b2ae to 4edb171 Compare August 31, 2023 23:05
@luca-montaigut luca-montaigut force-pushed the add-recurring-message-feature branch from b49ecc8 to 6448e24 Compare September 1, 2023 11:29
src/add-cron.ts Outdated
},
null,
true,
'Europe/Paris',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make it configurable

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how did you imagine that ? An optionnal subcommand ?

@luca-montaigut luca-montaigut changed the title feat: /cron command to add recurring messages feat: /recurrent command to add recurring messages Sep 3, 2023
@luca-montaigut luca-montaigut requested a review from potb September 3, 2023 16:58
@luca-montaigut luca-montaigut force-pushed the add-recurring-message-feature branch from d1464de to ddc502f Compare September 3, 2023 17:09
@luca-montaigut luca-montaigut force-pushed the add-recurring-message-feature branch from ddc502f to f4c2a04 Compare September 3, 2023 17:12
@luca-montaigut luca-montaigut deleted the add-recurring-message-feature branch September 4, 2023 15:27
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 this pull request may close these issues.

2 participants