Skip to content

Tidied up pipelines.js.es6 and removed jQuery where acceptable.

Luke "Jared" Bennett requested to merge tidy-up-pipeline-js into master

What does this MR do?

A week or so ago, I made some changes to pipeline.js as it was a single function and we now needed to expand on the JS logic for pipelines. I decided to turn it into class Pipelines of pipelines.js.es6. To avoid review confusion I held back from changing the pre-existing function so it dealt with instance members. Every time you click the button it would re-query all the elements it needed, now its a class it doesn't need that as we can store the required elements on construction.

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Merge request reports