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 alert status 'error' count and icon #1640

Merged
merged 1 commit into from
Jul 6, 2017

Conversation

moolitayer
Copy link

Show severity count for MiqAlertStatus with severity = 'alert' in the alerts dashboard
Before:
alerts_error_before

After:
alerts_error_after

@miq-bot
Copy link
Member

miq-bot commented Jul 2, 2017

Checked commit moolitayer@a68bdf8 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 👍

@moolitayer moolitayer changed the title Fix alert status 'error' count Fix alert status 'error' count and icon Jul 2, 2017
@moolitayer
Copy link
Author

@himdel please review

@moolitayer
Copy link
Author

@yaacov please review

Copy link
Member

@yaacov yaacov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@moolitayer
Copy link
Author

@himdel please take a look

@moolitayer
Copy link
Author

@dclarizio please review

@dclarizio dclarizio merged commit c0fbac3 into ManageIQ:master Jul 6, 2017
@dclarizio dclarizio added this to the Sprint 64 Ending Jul 10, 2017 milestone Jul 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants