- Issue created by @paucala
- @paucala opened merge request.
- @paucala opened merge request.
- Status changed to Needs work
over 1 year ago 11:40am 21 May 2023 - First commit to issue fork.
- Status changed to Needs review
over 1 year ago 1:45pm 21 May 2023 - 🇮🇳India bharath-kondeti Hyderabad
@BramDriesen Addressed review comments. Update the code with dependency injections. Please review
- 🇪🇸Spain paucala
I'm adding a new commit that includes more detailed info about errors, and deleting the errors_count as long as if we need to count the error we can do it latter.
- Status changed to Needs work
over 1 year ago 10:41am 24 May 2023 - Status changed to Needs review
over 1 year ago 11:36am 25 May 2023 - 🇪🇸Spain paucala
I've finished with the changes, i willl be testing it in the next days
- 🇪🇸Spain paucala
As you can see here, changes are working, we recive the status info on php, db, server and the last time cron was run
Not to hijack the current issue, but just thought I'd share my two cents, but I believe this would work better if we exposed some kind of API that lets contrib modules manipulate the status report results and add in their custom data to be used as they deem fit on the status_dashboard reports.
This way developers can provide site-specific information as they deem fit.
- 🇧🇪Belgium BramDriesen Belgium 🇧🇪
Yeah, I do agree. That would be the endgame for this module ;-)
- 🇬🇧United Kingdom ashlewis
Adding patch to fix error when used with associated status_dashboard module patch, as dashboard view expects last cron run to be timestamp (rather than string)
- 🇬🇧United Kingdom ashlewis
And here's one combined with the patch from here: https://www.drupal.org/project/status_dashboard_client/issues/3392852 🐛 False positive for security update when project has multiple supported versions Needs review
- 🇬🇧United Kingdom Rob Holmes
The patch below combines the patch from Ash Lewis with a new section that adds unsupported modules to the response.
- 🇧🇾Belarus aylis
Hello everyone!
To my thought, addition of php_version, last_cron_run, etc introduced in that issue are not this relevant for most of users.
I suggest everyone who now need additional info in response to use patch from this ✨ Add a module alter to amend the JSON response Active issue, probably combined with changes introduced in that module → . Thanks @rob holmes for both! - 🇧🇾Belarus aylis
I think further work and discussion on adding new information should be continued under this 📌 Incorporate Status Dashboard Client Extras Active issue, as Status Dashboard Client Extra module incorporates data that was suggested here to be added to Status Dashboard Client