Skip to content

Uniform CI status components in vue

Filipa Lacerda requested to merge 30286-ci-badge-component into master

What does this MR do?

Uniforms CI status in all vue components

  • Removes app/assets/javascripts/vue_shared/pipeline_svg_icons.js since it's repeated with app/assets/javascripts/vue_shared/ci_status_icons.js

  • Fixes css class being provided to CI icon

  • Creates CI Badge component to render CI Badge and merges

    • app/assets/javascripts/vue_shared/components/pipeline_status_icon.js
    • app/assets/javascripts/pipelines/components/status.js

Screen_Shot_2017-05-10_at_10.18.32

  • Uses CI badge component in all places it was repeated
  • Removes app/assets/javascripts/pipelines/components/stage.js since we already have app/assets/javascripts/pipelines/components/stage.vue

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes #30286 (closed)

Merge request reports

Loading