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.

Reply via email to