Skip to content

improve merge request widget status icon UX

Simon Knox requested to merge 37229-mr-widget-status-icon into master

What does this MR do?

Show failed icon if Pipeline failed

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

for specs, it is probably better to mock the computed properties but I did not know how
computed property logic could also afford some cleanup

Screenshots (if relevant)

Screen_Shot_2017-09-11_at_10.07.20_pm_copy

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes #37229 (closed)

Edited by Simon Knox

Merge request reports

Loading