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

Alert UI changes #3266

Merged
merged 3 commits into from Dec 30, 2016

Conversation

Projects
None yet
2 participants
@edmundoa
Member

edmundoa commented Dec 28, 2016

More alert UI refinements. Refs #3178

edmundoa added some commits Dec 28, 2016

Sort alert condition notifications by title
As we do in the alert notifications page.
Change alert condition navigation
Click on title to show details, as done in other pages.

@edmundoa edmundoa added this to the 2.2.0 milestone Dec 28, 2016

@bernd

bernd approved these changes Dec 30, 2016

LGTM 👍

@bernd bernd merged commit 85493ce into master Dec 30, 2016

4 checks passed

ci-web-linter Jenkins build graylog-pr-linter-check 1215 has succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
licence/cla Contributor License Agreement is signed.
Details

@bernd bernd deleted the alert-ui-changes branch Dec 30, 2016

@bernd bernd removed the ready-for-review label Dec 30, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment