Skip to content
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

[Security Solution][Preview] - Enable preview when only filters used #94018

Merged
merged 3 commits into from
Mar 9, 2021

Conversation

yctercero
Copy link
Contributor

@yctercero yctercero commented Mar 8, 2021

Summary

Addresses #93245 where the preview button was grayed out when only filters were present - no main query.

Adds unit tests.

Screen Shot 2021-03-08 at 2 13 28 PM

Screen Shot 2021-03-08 at 2 14 00 PM

Checklist

@yctercero yctercero added bug Fixes for quality problems that affect the customer experience release_note:fix v8.0.0 Team:Detections and Resp Security Detection Response Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. v7.13.0 labels Mar 8, 2021
@yctercero yctercero requested a review from a team as a code owner March 8, 2021 22:33
@yctercero yctercero self-assigned this Mar 8, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-solution (Team: SecuritySolution)

@elasticmachine
Copy link
Contributor

Pinging @elastic/security-detections-response (Team:Detections and Resp)

Copy link
Contributor

@dplumlee dplumlee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good 👍

eqlQueryLoading ||
isDisabled ||
query == null ||
(query != null && query.query.query === '' && query.filters.length === 0)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good ol query.query.query

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 7.8MB 7.8MB +101.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @yctercero

@yctercero yctercero merged commit e603bf5 into elastic:master Mar 9, 2021
yctercero added a commit to yctercero/kibana that referenced this pull request Mar 9, 2021
…lastic#94018)

### Summary

Addresses elastic#93245 where the preview button was grayed out when only filters were present - no main query.

# Conflicts:
#	x-pack/plugins/security_solution/public/detections/components/rules/query_preview/index.test.tsx
yctercero added a commit to yctercero/kibana that referenced this pull request Mar 9, 2021
…lastic#94018)

### Summary

Addresses elastic#93245 where the preview button was grayed out when only filters were present - no main query.

# Conflicts:
#	x-pack/plugins/security_solution/public/detections/components/rules/query_preview/index.test.tsx
yctercero added a commit that referenced this pull request Mar 9, 2021
…94018) (#94044)

### Summary

Addresses #93245 where the preview button was grayed out when only filters were present - no main query.

# Conflicts:
#	x-pack/plugins/security_solution/public/detections/components/rules/query_preview/index.test.tsx
yctercero added a commit that referenced this pull request Mar 9, 2021
…s used (#94018) (#94045)

* [Security Solution][Preview] - Enable preview when only filters used (#94018)

### Summary

Addresses #93245 where the preview button was grayed out when only filters were present - no main query.

# Conflicts:
#	x-pack/plugins/security_solution/public/detections/components/rules/query_preview/index.test.tsx

* fixing merge conflict
@yctercero yctercero deleted the preview_filter_only branch October 13, 2021 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience release_note:fix Team:Detections and Resp Security Detection Response Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. v7.13.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Security Solution] - Preview query grayed out when only filters present
4 participants