-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Show system status in Settings/General #27238
Conversation
@mmattel, thanks for your PR! By analyzing the history of the files in this pull request, we identified @bantu, @butonic and @DeepDiver1975 to be potential reviewers. |
Why did jenkins fail? |
Seems you added a new public method:
you need to add |
5ec2841
to
0616ac0
Compare
Tested, works. 👍 |
May I ask why the servername was not added to the admin pages as well? |
@Emil-G the admin page simply outputs whatever values are in status.php. If no server name is visible in status.php then it won't be visible there. But maybe you meant to add it as additional info on this page regardless of status.php ? (the intent here was only about status.php) |
@PVince81 yes, that is what I was asking. That would be useful admin info when servers are clustered. |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
@PVince81
Description
The status query is moved into a fuction making it available to be regualry called.
Printout of system details as you would see them like in status.php
Adding additional elements in the status query does not need additional code changes in the template.
Related Issue
#27231
Motivation and Context
This PR shows the elements like you see in ownclouddomain.com/status.php at the bottom of Settings/General
How Has This Been Tested?
call /settings/admin?sectionid=general and move to the bottom to see the result
Screenshots (if appropriate):
Types of changes
Checklist: