-
Notifications
You must be signed in to change notification settings - Fork 2k
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
System and sysbtach clients tab #11172
Conversation
lgfa29
commented
Sep 10, 2021
- Fix create, edit and status sorting
- Fix initial client list filtering to only include clients targeted by the job
- Make table row clickable
- Some code refactoring
Ember Asset Size actionAs of 8f711a4 Files that got Bigger 🚨:
Files that stayed the same size 🤷:
|
Ember Test Audit comparison
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
case 'running': | ||
statusSummary.runningAllocs++; | ||
break; | ||
case 'lost': | ||
statusSummary.lostAllocs++; | ||
break; | ||
case 'failed': | ||
statusSummary.failedAllocs++; | ||
break; | ||
case 'complete': | ||
statusSummary.completeAllocs++; | ||
break; | ||
case 'starting': | ||
statusSummary.startingAllocs++; | ||
break; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll probably refactor this once more to be an enum.
I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions. |