- 🇩🇪Germany jurgenhaas Gottmadingen
I've tested this just now and it works as expected. Not setting it to RTBC as for outstanding test coverage.
- 🇩🇪Germany c-logemann Frankfurt/M, Germany
I agree that we should increase test coverage. But in this case I think we don't need an extra test: "if disabled it's truly not running at cron".
- 🇩🇪Germany c-logemann Frankfurt/M, Germany
In any case it would be helpful if there is a merge request for this change.
- Merge request !49Issue #3313343: D8: Add option to disable cron activity → (Open) created by solideogloria
- last update
over 1 year ago 86 pass It works. Just wondering, should a message be logged when cron link checking is skipped?
- 🇩🇪Germany c-logemann Frankfurt/M, Germany
> should a message be logged when cron link checking is skipped?
I don't think so. Especially on functionality like linkchecker I like to control this directly with own cron jobs without extra modules. So this logging if someone like to add should also get an own config to disable.
- 🇦🇹Austria alexh
I can confirm that the patch is working.
On the other hand, I can also recommend to install Ultimate Cron → module, which provides an easy UI to schedule each module's cron run differently, e.g. linkchecker only once or several times in the night. No patch needed in that scenario. - Status changed to Needs work
5 months ago 7:15am 23 July 2024 - 🇦🇺Australia VladimirAus Brisbane, Australia
Thanks for your contribution.
Can we update the it to the latest branch? solideogloria → changed the visibility of the branch 2.0.x to hidden.
- Status changed to Needs review
5 months ago 2:28pm 23 July 2024 Also, the settings form in 2.x is broken.
🐛 Linkchecker settings form is broken Active
- 🇦🇺Australia VladimirAus Brisbane, Australia
VladimirAus → changed the visibility of the branch 3313343-disable-cron to hidden.
- Status changed to Needs work
5 months ago 2:20pm 24 July 2024 - 🇦🇺Australia VladimirAus Brisbane, Australia
hook_update
for new config property?
- Status changed to Needs review
5 months ago 4:43pm 24 July 2024