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

Sort reports alphabetically #2565

Merged
merged 2 commits into from
Oct 31, 2017

Conversation

yrudman
Copy link
Contributor

@yrudman yrudman commented Oct 30, 2017

Fixing report sorting: reports are not sorted, sorting order in navigation tree is not consistent and different than in table
Issue: ManageIQ/manageiq#16083

BEFORE:
before

AFTER:
after

@miq-bot add-label enhancement, reporting

@miq-bot
Copy link
Member

miq-bot commented Oct 30, 2017

@yrudman Cannot apply the following label because they are not recognized: reporting

@miq-bot
Copy link
Member

miq-bot commented Oct 30, 2017

Checked commits yrudman/manageiq-ui-classic@4de20f4~...8c2cb8b with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. 🏆

@yrudman
Copy link
Contributor Author

yrudman commented Oct 30, 2017

\cc @@h-kataria

@h-kataria
Copy link
Contributor

looks good, can be merged when green

@chessbyte chessbyte closed this Oct 31, 2017
@chessbyte chessbyte reopened this Oct 31, 2017
@chessbyte chessbyte merged commit e41ecad into ManageIQ:master Oct 31, 2017
@chessbyte chessbyte self-assigned this Oct 31, 2017
@chessbyte chessbyte added this to the Sprint 72 Ending Oct 30, 2017 milestone Oct 31, 2017
@yrudman yrudman deleted the sort-reports-alphabetically branch October 31, 2017 13:40
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