-
Notifications
You must be signed in to change notification settings - Fork 297
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
Fixes #35595 - Add content profile to selectable columns #10305
Fixes #35595 - Add content profile to selectable columns #10305
Conversation
Issues: #35595 |
14272f3
to
b4b4e52
Compare
Thanks, @adamruzicka, it was just a copy-paste typo :( this column can't be sortable. Also, here is a screenshot to make the initial review easier: |
Code looks fine, starting to test |
@ofedoren what was your hammer command for the table preference columns? |
hammer user table-preference create --name hosts --columns |
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.
ACK, tested each column and it looks correct. @adamruzicka has all of your concerns been addressed?
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.
Yes, they were, let's get this in
What are the changes introduced in this pull request?
Coulmns from
Content Hosts
page are added as selectable columns toHosts
page.Considerations taken when implementing this change?
Basically it allows to proceed with merging
Content Hosts
page intoHosts
page.What are the testing steps for this pull request?
Currently, the only way to see the columns is to use
hammer user table-preference
command to create a preference forhosts
table with certain columns. Core's and newContent
profiles can be used. Until theforeman/foreman#9323 is done, there is no other way to select the whole profile.You can also use theforeman/foreman#9323 to test the feature out as well as provide some feedback while it's still in early stage.