Problem/Motivation

Valuable data from ohDear app could be shown from inside the drupal and that way site users could have access to that data.

List of broken links, downtime reports and performance graphs could be shown on a new report page.

As a model we could check a plugin for wordpress: https://wordpress.org/plugins/ohdear/

Issue fork ohdear_integration-3314104

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

useernamee created an issue. See original summary.

useernamee’s picture

useernamee commented
Priority: Normal » Minor
useernamee’s picture

useernamee commented

From the documentation:
* Broken links
* Downtime reports
* Performance

useernamee opened merge request !11

useernamee’s picture

useernamee commented
Assigned: Unassigned » useernamee
useernamee’s picture

useernamee commented
Status: Active » Needs review

I've provided controller and drush commands for OhDear checks, uptime and broken links.

This PR is already quite big, so as a follow up I'd recommend to add error checking (for missing site id) and to make results prettier (Successful checks should be green, etc).

useernamee’s picture

useernamee commented
Assigned: useernamee » Unassigned
useernamee’s picture

useernamee commented
Status: Needs review » Needs work

Tests are now failing.

useernamee’s picture

useernamee commented
Status: Needs work » Needs review
roderik’s picture

roderik
Primary language Dutch
Location Amsterdam,NL / Budapest,HU
commented
Status: Needs review » Needs work

Reviewed without testing.

useernamee’s picture

useernamee commented
Priority: Minor » Normal
Status: Needs work » Needs review

I've fixed the sloppiness @roderik pointed out in PR.

This PR will also break d8 & d9 compatibility so we'll probably release it as a major version.

roderik’s picture

roderik
Primary language Dutch
Location Amsterdam,NL / Budapest,HU
commented
Status: Needs review » Reviewed & tested by the community

Looks good to me. I see you've done more cleanup than I'd spotted.

useernamee’s picture

useernamee commented
Version: 1.x-dev » 2.x-dev

  • useernamee committed 4e29075c on 2.x 
    Issue #3314104 by useernamee, roderik: Show most valueable data from...
useernamee’s picture

useernamee commented
Status: Reviewed & tested by the community » Fixed

Merged in 2.x-dev branch.

useernamee’s picture

useernamee commented

Released in 2.0.0

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.