Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: template status bars had failed and terminated swapped visually #140

Merged
merged 1 commit into from Jul 16, 2020

Conversation

Vafilor
Copy link
Contributor

@Vafilor Vafilor commented Jul 14, 2020

What this PR does:

Fixes an issue where the status bars for Workflow Template page had failed and terminated swapped.

Which issue(s) this PR fixes:

Fixes onepanelio/onepanel#408

Special notes for your reviewer:

To test this easier, you can modify this line

https://github.com/onepanelio/core-ui/blob/master/src/app/workflow-template/workflow-template-status/workflow-template-status.component.ts#L75

change it to

this.values = [5, 6, 7, 8];

this allows you to see what the different bars would look like

@Vafilor Vafilor requested a review from rushtehrani July 14, 2020 19:19
@rushtehrani rushtehrani merged commit 0e5e0f7 into dev Jul 16, 2020
@rushtehrani rushtehrani deleted the fix/onepanelio.core.408-status.bar.order branch July 16, 2020 17:11
@rushtehrani rushtehrani added this to the v0.12.0 milestone Jul 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants