- Issue created by @davisben
- last update
about 1 year ago 3 pass, 2 fail - @davisben opened merge request.
- Status changed to Needs review
about 1 year ago 6:33pm 10 October 2023 - last update
about 1 year ago 11 pass - Status changed to Needs work
about 1 year ago 4:47pm 21 October 2023 - π·πΊRussia jureth Omsk
1. Why there's a check for ScheduledCronServiceInterface in getLastExecutionTime() ? As I understand the last time value is stored for every handler.
2. Technically, 0 is valid unix timestamp, so I'm not sure why getLastExecutionTime() shouldn't return null in case the value is missing. - last update
about 1 year ago 11 pass - last update
about 1 year ago 11 pass - Status changed to Needs review
about 1 year ago 4:31pm 27 October 2023 - πΊπΈUnited States davisben NH
I addressed the concerns in #4, and rebased against 2.0.x.
- last update
about 1 year ago 11 pass - πΊπ¦Ukraine vasyl.kletsko
vasyl.kletsko β made their first commit to this issueβs fork.
-
vasyl.kletsko β
committed d36d717e on 2.0.x authored by
davisben β
Issue #3393041 by davisben, jureth: Store last execution time
-
vasyl.kletsko β
committed d36d717e on 2.0.x authored by
davisben β
- Status changed to Fixed
about 1 year ago 8:41am 15 November 2023 Automatically closed - issue fixed for 2 weeks with no activity.