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 #14536: Add a button to show/hide Directives and Groups tree in Rule details #2168
Conversation
This is really, really good but I think we need a little bit more information on the directive/group before merging, at least link toward said objects (and perhaps description/ policy mode). |
8243c72
to
c503ddc
Compare
Commit modified |
for (var o in $scope.directives) { | ||
arr.push({"id":o, "name": $scope.directives[o] }); | ||
} | ||
for (var o in $scope.directives) { |
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.
It seems that that part is not working as expected. Directive can't be selected / unselected (groups work)
c503ddc
to
431b8aa
Compare
PR rebased |
431b8aa
to
29169b7
Compare
Commit modified |
29169b7
to
77cb5e6
Compare
Commit modified |
Perfect! |
OK, merging this PR |
https://issues.rudder.io/issues/14536