Closed (fixed)
Project:
Git Info Report
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
14 Oct 2019 at 11:14 UTC
Updated:
7 Jun 2020 at 05:09 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
lolandese commentedWe could also make it a local tab of the Status Report.
Comment #3
lolandese commentedComment #4
lolandese commentedEasiest and probably also best (most straightforward) is to strip down gitinfo.install. We add also
$phase === 'install'so module install will fail if:The rest can then be moved to a separate dedicated page but leaving the built-in check to avoid breaking.
It might be good to keep a link to the Git Info report. Rename to just "Git info".
Comment #5
lolandese commentedAnother advantage of this approach is that we now can give permissions to any role to view Git info, which is pretty harmless. Furthermore, we have now more horizontal screen space available as we do not use the left column for report sections anymore.
To see how to replicate the functionality of the Status report to have warnings places separately at the top.
Comment #6
lolandese commentedThe attached patch only leaves the Git version info on the Status report with a link to the detailed report that now lives at admin/reports/gitinfo. We now can give permissions to any role to view Git info.
Comment #7
lolandese commented\Drupal calls should be avoided in classes, use dependency injection insteadConfiguration for the Git Info
section in the Status reportReport.Connect with remote repos
and check if your Git version is up to dateComment #8
lolandese commentedUsing Drupal status messages for errors, warnings, and OK.
Comment #10
lolandese commented