Branch: refs/heads/master
Home: https://github.com/jenkinsci/jenkins
Commit: 44c59a6f5ce805d690c75253ebf34c3983b090c4
https://github.com/jenkinsci/jenkins/commit/44c59a6f5ce805d690c75253ebf34c3983b090c4
Author: Félix Queiruga <[email protected]>
Date: 2020-11-26 (Thu, 26 Nov 2020)
Changed paths:
A core/src/main/java/jenkins/management/AdministrativeMonitorsApi.java
A core/src/main/java/jenkins/management/AdministrativeMonitorsApiData.java
M core/src/main/java/jenkins/management/AdministrativeMonitorsDecorator.java
A
core/src/main/resources/jenkins/management/AdministrativeMonitorsApiData/monitorsList.jelly
A
core/src/main/resources/jenkins/management/AdministrativeMonitorsApiData/monitorsList.properties
A
core/src/main/resources/jenkins/management/AdministrativeMonitorsApiData/monitorsList_bg.properties
A
core/src/main/resources/jenkins/management/AdministrativeMonitorsApiData/monitorsList_de.properties
A
core/src/main/resources/jenkins/management/AdministrativeMonitorsApiData/monitorsList_it.properties
A
core/src/main/resources/jenkins/management/AdministrativeMonitorsApiData/monitorsList_pl.properties
M
core/src/main/resources/jenkins/management/AdministrativeMonitorsDecorator/footer.jelly
M
core/src/main/resources/jenkins/management/AdministrativeMonitorsDecorator/resources.css
M
core/src/main/resources/jenkins/management/AdministrativeMonitorsDecorator/resources.js
A
test/src/test/java/jenkins/management/AdministrativeMonitorsDecoratorTest.java
Log Message:
-----------
Load admin monitors popup content via ajax (#5063)
* Admin monitors popup is now loaded through an API
* Fixes form submission inside the popup
- Wraps it in an l:ajax template so that proper variables are set
- Applies the csrf crumbs to the forms after the popup is constructed
* Handle authorization and empty state
* Apply suggestions from code review
Co-authored-by: Tim Jacomb <[email protected]>
* Fix JS error where there were monitors of only one kind
* Revert changes to indentation of inline JS
* Clarify ESC keyCode with a comment
* Update
core/src/main/resources/jenkins/management/AdministrativeMonitorsDecorator/resources.js
Co-authored-by: Tim Jacomb <[email protected]>
* Multiple small adjustments
- Remove unused imports
- Remove non-necessary call for the response that are set in the getView
- Keep the previous improvement about limiting the number of call to
getMonitorList()
Co-authored-by: Tim Jacomb <[email protected]>
Co-authored-by: Wadeck Follonier <[email protected]>
--
You received this message because you are subscribed to the Google Groups
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/jenkinsci-commits/jenkinsci/jenkins/push/refs/heads/master/a14b68-44c59a%40github.com.