-
Notifications
You must be signed in to change notification settings - Fork 73
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 #8563: Make the search bar in menu able to also search rule, group, directive, etc #1119
Conversation
PR rebased |
c79dee1
to
48be9ed
Compare
// only search if query is on Directives and attributes contains | ||
// DirectiveId, DirectiveVarName, DirectiveVarValue, TechniqueName, TechniqueVersion | ||
|
||
val attributes: Set[QSAttribute] = query.attributes.intersect(QSObject.Directive.attributes) |
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.
QSObject.Directive can be replaced by QSDirective
PR rebased |
4400edb
to
8accb18
Compare
PR rebased |
8accb18
to
f1658ad
Compare
* (just enabled/disabled with the parsing) | ||
*/ | ||
sealed trait FeatureSwitch { def name: String } | ||
object FeatureSwitch { |
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.
Use the implementation from PR #1141
PR rebased |
c1245c2
to
08b68e1
Compare
PR rebased |
1 similar comment
PR rebased |
08b68e1
to
610db0c
Compare
PR rebased |
610db0c
to
743fe2e
Compare
PR rebased |
743fe2e
to
5d7083c
Compare
PR rebased |
5d7083c
to
75fa5f2
Compare
…up, directives...
PR rebased |
75fa5f2
to
11a3a35
Compare
PR rebased |
Replaced by #1188 |
https://www.rudder-project.org/redmine/issues/8563
This is a work in progress with the current prototype, to allow some pre-validation (try the feature by yourself!) and looking to the code for architecture remarks.