• jenkins-2.320

      (For details, please refer to the GDoc link in the Epic description)

      Difficulty: easy
      Skill requirement: JavaScript (basic), Jelly (basic)

      Move the script tag BuildTimelineWidget/control.jelly#L40-L167 to its own file.

      To inject separate file into Jelly, you can use <st:adjunct includes=...
      One variable injected, you will need to use advice from https://www.jenkins.io/doc/developer/security/xss-prevention/#passing-values-to-javascript

          [JENKINS-66718] Un-inlining BuildTimelineWidget/control.jelly

          Merged, will be released tomorrow normally

          Wadeck Follonier added a comment - Merged, will be released tomorrow normally

          Good job, your contribution is now part of a weekly release (jenkins-2.320)

          Wadeck Follonier added a comment - Good job, your contribution is now part of a weekly release (jenkins-2.320)

            megathaum megathaum
            wfollonier Wadeck Follonier
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: