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

Merge telegram and twitter actions #22

Closed
SirusCodes opened this issue Dec 27, 2021 · 1 comment · Fixed by #108
Closed

Merge telegram and twitter actions #22

SirusCodes opened this issue Dec 27, 2021 · 1 comment · Fixed by #108
Assignees
Labels
CI/CD Issues related to CI/CD enhancement New feature or request good first issue Good for newcomers

Comments

@SirusCodes
Copy link
Member

Currently, we have actions as:

  • Reminder
  1. Telegram
  2. Twitter
  • Catchup
  1. Telegram
  2. Twitter

This costs a total of 4 actions which could be done in 2.
So the proposal is to merge all the reminder actions in 1 and catchup in 1.

@SirusCodes SirusCodes added good first issue Good for newcomers CI/CD Issues related to CI/CD labels Dec 27, 2021
@HarshKapadia2 HarshKapadia2 added the enhancement New feature or request label Dec 27, 2021
@HarshKapadia2
Copy link
Member

CRON job timings for the reminder workflow should be 20 16 * * 6 and for the CatchUp workflow should be 30 11 * * 6.

Please keep the Twitter job above the Telegram job in each workflow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD Issues related to CI/CD enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants