- First commit to issue fork.
- @acbramley opened merge request.
- Status changed to Needs work
about 2 years ago 1:36am 11 February 2023 - πΊπΈUnited States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request β as a guide.
Appears the MR has test failures.
Have not yet reviewed.
- Status changed to Needs review
about 2 years ago 2:43am 21 February 2023 - π¦πΊAustralia acbramley
I'm not sure what the old logs were before (with
$this->anything()
coming first) but now we just have our log and then the cron run completed notice. - Status changed to Needs work
about 2 years ago 6:23pm 21 February 2023 - πΊπΈUnited States smustgrave
Thanks.
Tagged for a change record to announce this new behavior.
The code changes look good to me so +1 there.
- Status changed to Needs review
about 2 years ago 9:51pm 21 February 2023 - π¦πΊAustralia acbramley
- Status changed to RTBC
about 2 years ago 10:42pm 21 February 2023 - πΊπΈUnited States smustgrave
perfect and adding the example really helped!
- Status changed to Fixed
about 2 years ago 9:50am 22 February 2023 Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Fixed
over 1 year ago 12:24pm 8 September 2023