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

Unify sorting in GOD of Button Groups #5513

Merged
merged 1 commit into from
May 10, 2019

Conversation

ZitaNemeckova
Copy link
Contributor

GOD tree sorts Button Groups by their name. Fixing table to have same sort as tree.

Go to Automation -> Automate -> Generic Objects -> Add few Button Groups with names that go from Z to A (so their chronological and alphabetical orders are different!) -> see tree and table orders are different

Before:
image

After:
image

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1665031

#5146 didn't fix the problem completely.

@miq-bot add_label bug, generic objects, automate/automation, hammer/yes, gaprindashvili/yes, bugzilla needed

GOD tree sorts Button Groups by their name. Fixing table to have same sort as tree

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1665031
@miq-bot
Copy link
Member

miq-bot commented May 1, 2019

@ZitaNemeckova Cannot apply the following label because they are not recognized: automate/automation

@ZitaNemeckova
Copy link
Contributor Author

@miq-bot add_label automation/automate

@miq-bot
Copy link
Member

miq-bot commented May 1, 2019

Checked commit ZitaNemeckova@578acdb with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. 🏆

@ZitaNemeckova
Copy link
Contributor Author

@miq-bot add_label changelog/yes

@ZitaNemeckova
Copy link
Contributor Author

@miq-bot remove_label bugzilla needed

@ZitaNemeckova
Copy link
Contributor Author

@romanblanco please have a look, thanks :)

Copy link
Member

@romanblanco romanblanco left a comment

Choose a reason for hiding this comment

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

Tesed, works 👍

@martinpovolny martinpovolny merged commit 26490f9 into ManageIQ:master May 10, 2019
@martinpovolny martinpovolny added this to the Sprint 111 Ending May 13, 2019 milestone May 10, 2019
@martinpovolny martinpovolny self-assigned this May 10, 2019
@ZitaNemeckova ZitaNemeckova deleted the god_fix branch May 31, 2019 12:27
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