Skip to content
This repository has been archived by the owner on Aug 27, 2022. It is now read-only.

Feature: Debug Panel #275

Merged
merged 6 commits into from
Jul 30, 2021
Merged

Feature: Debug Panel #275

merged 6 commits into from
Jul 30, 2021

Conversation

jacques42
Copy link
Collaborator

@jacques42 jacques42 commented Jul 23, 2021

Prerequisites checklist

What is the purpose of this pull request? (put an "x" next to an item)

  • New feature

What changes did you make? (Give an overview)

Implements a panel for to help debugging in case of issues. Focus is to be able to access through the browser key configuration and log files on the server side. This feature is 1) for fast and efficient debugging iterations and 2) also well positioned to help people with less experience on the server administration and Unix / Raspberry Pi OS side of things.

Access to the debug panel is available through the admin panel (switch to expert view) or via direct URL http://localhost/admin/debugpanel.php

grafik

In particular useful for looking at log-files during operations, the debug panel allows to activate auto refresh. This feature will reload current content every 1 second and also automatically scroll to the bottom of the content to see what has changed.

The debug panel is architected in a way that additional debug items can easily be added

UI : admin/debugpanel.php
EXE: api/serverInfo.php

Is there anything you'd like reviewers to focus on?

@jacques42 jacques42 requested a review from andi34 July 23, 2021 14:19
@jacques42 jacques42 self-assigned this Jul 23, 2021
api/serverInfo.php Outdated Show resolved Hide resolved
api/serverInfo.php Outdated Show resolved Hide resolved
Change-Id: I18d134e9900f2a99234d020bbece1ff3c06f5834
Change-Id: I36e8005794acdf90c245cbb0bdb011df0b0401fc
@andi34 andi34 force-pushed the v3/feature/debugpanel branch from 16ecb43 to 9f9508d Compare July 24, 2021 09:44
Copy link
Owner

@andi34 andi34 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use the dumpfile function?

api/serverInfo.php Outdated Show resolved Hide resolved
api/serverInfo.php Outdated Show resolved Hide resolved
@andi34 andi34 mentioned this pull request Jul 29, 2021
2 tasks
api/serverInfo.php Outdated Show resolved Hide resolved
Change-Id: Ib619c7f43902c08db2bcd38cafb68b5018d743f8
@andi34
Copy link
Owner

andi34 commented Jul 29, 2021

Looks good and great idea! Thanks!
Good to go now from my side. Not sure if you like to integrate the changes from #277 here too?

@jacques42 jacques42 merged commit 3f4e8ba into dev Jul 30, 2021
@jacques42 jacques42 deleted the v3/feature/debugpanel branch July 30, 2021 10:57
echo '<li><a class="adminnavlistelement" href="#myconfig" id="nav-myconfig"><div><span data-i18n="myconfig">myconfig</span></div></a></li>';
echo '<li><a class="adminnavlistelement" href="#remotebuzzerlog" id="nav-remotebuzzerlog"><div><span data-i18n="remotebuzzer">remotebuzzer</span></div></a></li>';
echo '<li><a class="adminnavlistelement" href="#synctodrivelog" id="nav-synctodrivelog"><div><span data-i18n="synctodrive">synctodrive</span></div></a></li>';
echo '<li><a class="adminnavlistelement" href="#cameralog" id="nav-cameralog"><div><span data-i18n="cameralog">cameralog</span></div></a></li>';
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tab vs spaces

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants