Skip to content

Conversation

@thassiov
Copy link
Contributor

@thassiov thassiov commented Mar 29, 2021

Proposed changes (including videos or screenshots)

Create and schedule a task manually at scheduleRecurring method so the first iteration runs after the configured interval. This is accomplished by adding the setting skipImmediate: true when setting up the task.

Issue(s)

Steps to test or reproduce

Further comments

Added a lot of documentation and make the module log errors properly using the official logger

@thassiov thassiov marked this pull request as ready for review March 29, 2021 20:03
@thassiov thassiov requested review from d-gubert and shiqimei March 29, 2021 20:03
@thassiov thassiov changed the title [FIX][APPS] First iteration of recurring tasks does not repect given interval [FIX][APPS] First iteration of recurring tasks does not respect given interval Mar 30, 2021
@thassiov
Copy link
Contributor Author

Depends on RocketChat/Rocket.Chat.Apps-engine#403

@thassiov thassiov requested a review from d-gubert April 12, 2021 13:43
shiqimei
shiqimei previously approved these changes Apr 12, 2021
@d-gubert d-gubert modified the milestones: 3.13.2, 3.14.0 Apr 15, 2021
d-gubert
d-gubert previously approved these changes Apr 15, 2021
@d-gubert d-gubert changed the title [FIX][APPS] First iteration of recurring tasks does not respect given interval [IMPROVE][APPS] Scheduler option to skip immediate execution of recurring jobs Apr 19, 2021
@sampaiodiego sampaiodiego merged commit bf83826 into develop Apr 21, 2021
@sampaiodiego sampaiodiego deleted the fix/apps-scheduler-recurring-immediate branch April 21, 2021 05:15
@sampaiodiego sampaiodiego mentioned this pull request Apr 28, 2021
@sampaiodiego sampaiodiego mentioned this pull request May 28, 2021
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.

5 participants