- π³πΏNew Zealand quietone
This is suitable for a first time contributor, leaving novice ag.
- First commit to issue fork.
- Merge request !9497Issue #1630576: Moved cron completion watchdog message from ::setCronLastTime... β (Closed) created by nexusnovaz
- Status changed to Needs review
3 months ago 11:15am 15 September 2024 - π¬π§United Kingdom nexusnovaz
Hey, this is ready for review, please see !9497.
I moved the watchdog message to just before
$return = TRUE;
as to not confuse failed crons (due to cron lock, etc.). However, this can be moved again if needed.Thanks
- Status changed to RTBC
3 months ago 1:42pm 16 September 2024 - π¬π§United Kingdom catch
Looks good. Committed/pushed to 11.x and cherry-picked back through to 10.3.x, thanks!
- Status changed to Fixed
3 months ago 8:13pm 16 September 2024 Automatically closed - issue fixed for 2 weeks with no activity.