-
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
Display only manually added my company tags on topology screens #3007
Display only manually added my company tags on topology screens #3007
Conversation
Checked commit skateman@4bf2e39 with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0 |
tests please :) |
@ohadlevy this is being tested in the model, see the linked PR above... |
btw: off topic - is there an easy way to zoom into the topology view? while looking at https://github.com/ansible/awx I saw they have something similar that represent workflows that you can zoom into and take actions in that context, maybe worth looking into that |
@ohadlevy work in progress patternfly/angular-patternfly#633 |
@miq-bot remove_label pending core |
Display only manually added my company tags on topology screens (cherry picked from commit da363ba) https://bugzilla.redhat.com/show_bug.cgi?id=1526049
Gaprindashvili backport details:
|
When building the JSON for a topology, all the related tags are being fetched. This is a problem as we don't want to expose internal tags in the UI. As a 🎄 🎁 @lpichler added a new method into
ActsAsTaggable
that fetches only the tags we need.Before:
After:
@miq-bot add_label topology, bug, pending core, gaprindashvili/yes
https://bugzilla.redhat.com/show_bug.cgi?id=1519457