Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-69031

f:repeatable error in Elasticbox plugin (regression in 2.264)

      While trying to understand more completely the usage of YAHOO.util.Connect (for PR-6840), I was looking at variables.js (thanks Basil for the pointer).
      I was not able to configure the plugin correctly as the repeatable agent configuration was broken. It seems that the placeholder content is displayed but it's not parsed when sent to the server and there is no way to add new ones. I imagine it's just because of the JS error that the form is not rendered correctly.

      I tested with 2.359, it's broken. When tested with 2.222.4, the plugin (with same version) was working fine, meaning it's highly likely that the root cause is about a recent UI change.

          [JENKINS-69031] f:repeatable error in Elasticbox plugin (regression in 2.264)

          Wadeck Follonier created issue -
          Wadeck Follonier made changes -
          Description Original: While trying to understand more completely the usage of YAHOO.util.Connect (for [https://github.com/jenkinsci/jenkins/pull/6840|PR-6840]), I was looking at [https://github.com/jenkinsci/elasticbox-plugin/blob/master/src/main/webapp/js/variables.js#L591-L605|variables.js] (thanks Basil for the pointer).
          I was not able to configure the plugin correctly as the [https://github.com/jenkinsci/elasticbox-plugin/blob/6a387a2321219213b521cb4b8448292654105726/src/main/resources/com/elasticbox/jenkins/ElasticBoxCloud/config.jelly#L41-L43|repeatable agent configuration] was broken. It seems that the placeholder content is displayed but it's not parsed when sent to the server and there is no way to add new ones. I imagine it's just because of the JS error that the form is not rendered correctly.

          I tested with 2.359, it's broken. When tested with 2.222.4, the plugin (with same version) was working fine, meaning it's highly likely that the root cause is about a recent UI change.
          New: While trying to understand more completely the usage of YAHOO.util.Connect (for [PR-6840|https://github.com/jenkinsci/jenkins/pull/6840]), I was looking at [https://github.com/jenkinsci/elasticbox-plugin/blob/master/src/main/webapp/js/variables.js#L591-L605|variables.js] (thanks Basil for the pointer).
          I was not able to configure the plugin correctly as the [https://github.com/jenkinsci/elasticbox-plugin/blob/6a387a2321219213b521cb4b8448292654105726/src/main/resources/com/elasticbox/jenkins/ElasticBoxCloud/config.jelly#L41-L43|repeatable agent configuration] was broken. It seems that the placeholder content is displayed but it's not parsed when sent to the server and there is no way to add new ones. I imagine it's just because of the JS error that the form is not rendered correctly.

          I tested with 2.359, it's broken. When tested with 2.222.4, the plugin (with same version) was working fine, meaning it's highly likely that the root cause is about a recent UI change.
          Wadeck Follonier made changes -
          Description Original: While trying to understand more completely the usage of YAHOO.util.Connect (for [PR-6840|https://github.com/jenkinsci/jenkins/pull/6840]), I was looking at [https://github.com/jenkinsci/elasticbox-plugin/blob/master/src/main/webapp/js/variables.js#L591-L605|variables.js] (thanks Basil for the pointer).
          I was not able to configure the plugin correctly as the [https://github.com/jenkinsci/elasticbox-plugin/blob/6a387a2321219213b521cb4b8448292654105726/src/main/resources/com/elasticbox/jenkins/ElasticBoxCloud/config.jelly#L41-L43|repeatable agent configuration] was broken. It seems that the placeholder content is displayed but it's not parsed when sent to the server and there is no way to add new ones. I imagine it's just because of the JS error that the form is not rendered correctly.

          I tested with 2.359, it's broken. When tested with 2.222.4, the plugin (with same version) was working fine, meaning it's highly likely that the root cause is about a recent UI change.
          New: While trying to understand more completely the usage of YAHOO.util.Connect (for [PR-6840|https://github.com/jenkinsci/jenkins/pull/6840]), I was looking at [variables.js|https://github.com/jenkinsci/elasticbox-plugin/blob/master/src/main/webapp/js/variables.js#L591-L605] (thanks Basil for the pointer).
          I was not able to configure the plugin correctly as the [repeatable agent configuration|https://github.com/jenkinsci/elasticbox-plugin/blob/6a387a2321219213b521cb4b8448292654105726/src/main/resources/com/elasticbox/jenkins/ElasticBoxCloud/config.jelly#L41-L43] was broken. It seems that the placeholder content is displayed but it's not parsed when sent to the server and there is no way to add new ones. I imagine it's just because of the JS error that the form is not rendered correctly.

          I tested with 2.359, it's broken. When tested with 2.222.4, the plugin (with same version) was working fine, meaning it's highly likely that the root cause is about a recent UI change.
          Daniel Beck made changes -
          Labels Original: regression ui New: regression tables-to-divs-regression ui ux
          Daniel Beck made changes -
          Summary Original: [core] f:repeatable error in Elasticbox plugin New: [core] f:repeatable error in Elasticbox plugin (regression in 2.264)
          Tim Jacomb made changes -
          Component/s New: elasticbox-plugin [ 19022 ]
          Component/s Original: core [ 15593 ]
          Tim Jacomb made changes -
          Summary Original: [core] f:repeatable error in Elasticbox plugin (regression in 2.264) New: f:repeatable error in Elasticbox plugin (regression in 2.264)
          Tim Jacomb made changes -
          Labels Original: regression tables-to-divs-regression ui ux New: regression tables-to-divs-regression

            Unassigned Unassigned
            wfollonier Wadeck Follonier
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: