-
Notifications
You must be signed in to change notification settings - Fork 357
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
Add Tenant relationship in Providers, Automate Domain and Services #5575
Add Tenant relationship in Providers, Automate Domain and Services #5575
Conversation
@miq-bot add_label changelog/yes |
270ff0d
to
e4a2688
Compare
4203c38
to
3d2c7a5
Compare
3d2c7a5
to
2363090
Compare
@miq-bot add_reviewer @himdel |
0507022
to
735e34c
Compare
735e34c
to
132e9a2
Compare
132e9a2
to
9f6db06
Compare
@@ -30,13 +30,17 @@ cols: | |||
- total_storages | |||
- total_vms | |||
- total_miq_templates | |||
- tenant.name |
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.
The field should be either in cols
or under include
, not both.
(so, just include, col_order and headers for this one)
(same in the other yaml)
a761f39
to
244022d
Compare
Also fix existing failing spec and make small improvements.
244022d
to
3cd7de8
Compare
Checked commits hstastna/manageiq-ui-classic@5d3dd81~...3cd7de8 with ruby 2.3.3, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0 **
|
The part that is "pending core" was moved to a separate PR. |
RFE:
https://bugzilla.redhat.com/show_bug.cgi?id=1678122
More:
https://docs.google.com/document/d/1rpgBQA_MAfqZY00dnbRLMiOzqhbrB60UdnvSgg7X--Y/edit
What:
This PR adds displaying Tenant name in Providers, Automate Domain and Services. Tenant name will be displayed with this PR in Summary View of cloud or infra provider, in Relationships table. Also a new column with the Tenant name will be displayed in the List view of a Provider, Automate Domain or a Service Catalog Item/Catalog Bundle (it will not be included in: Grid View, Tile View, Dashboard view).
How to test:
Compute -> Infrastructure/Clouds -> Provider
and choose some provider if you want to check the Summary view
Automation -> Automate -> Explorer
_Services -> Catalogs -> Catalog Items -> choose some Catalog Item/Catalog Bundle
TODO:
display Tenant name in Summary View of any provider, in Relationships tabledisplay Tenant name as a new column in list view of providers(we need Hide columns in GTL lists for non-admin users(not only) #5616 and Modify method to hide columns dynamically in GTL list manageiq#18812)
add column Tenant name for Automate Domainadd column Tenant name for Service Catalog Items/Catalog Bundles (Summary screens)specsNote:
Tenant names are displayed only for admin users.
After:
Cloud Provider Summary View:
Infrastructure Provider Summary View:
Cloud Providers' List View:
Infra Providers' List View:
Automate Domain:
Service Catalog Item - Summary screen:
Service Catalog Bundle - Summary screen: