The TagsFilterDelegate needs to allow for OR to match on any instances so users can query local sources and remote sources that don't have metacard-tags, like cached federated data, at the same time.
Who is reviewing it?
@jlcsmith
@glenhein
Select relevant component teams:
@codice/core-apis
Ask 2 committers to review/merge the PR and tag them here.
@clockard
@andrewkfiedler
How should this be tested?
Any background context you want to provide?
What are the relevant tickets?
Fixes: #____
Screenshots
Checklist:
[ ] Documentation Updated
[ ] Update / Add Threat Dragon models
[ ] Update / Add Unit Tests
[ ] Update / Add Integration Tests
Notes on Review Process
Please see Notes on Review Process for further guidance on requirements for merging and abbreviated reviews.
Review Comment Legend:
✏️ (Pencil) This comment is a nitpick or style suggestion, no action required for approval. This comment should provide a suggestion either as an in line code snippet or a gist.
❓ (Question Mark) This comment is to gain a clearer understanding of design or code choices, clarification is required but action may not be necessary for approval.
❗ (Exclamation Mark) This comment is critical and requires clarification or action before approval.
What does this PR do?
This is a forward port of PR #6481
The TagsFilterDelegate needs to allow for OR to match on any instances so users can query local sources and remote sources that don't have metacard-tags, like cached federated data, at the same time.
Who is reviewing it?
@jlcsmith @glenhein
Select relevant component teams:
@codice/core-apis
Ask 2 committers to review/merge the PR and tag them here.
@clockard @andrewkfiedler
How should this be tested?
Any background context you want to provide?
What are the relevant tickets?
Fixes: #____
Screenshots
Checklist:
Notes on Review Process
Please see Notes on Review Process for further guidance on requirements for merging and abbreviated reviews.
Review Comment Legend: