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

Fix UI issues #1107

Merged
merged 4 commits into from
Aug 10, 2024
Merged

Fix UI issues #1107

merged 4 commits into from
Aug 10, 2024

Conversation

amsiglan
Copy link
Collaborator

@amsiglan amsiglan commented Aug 9, 2024

Description

This PR fixes

  • Page crash when clicking on a correlation graph node to get more details
  • Help text for data source selection controls
  • Pagination for IOCs inside threat intel source
  • Filtering of alerts for correlations

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Copy link

codecov bot commented Aug 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 28.11%. Comparing base (6439704) to head (e5eea0e).
Report is 18 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1107      +/-   ##
==========================================
- Coverage   31.76%   28.11%   -3.65%     
==========================================
  Files         162      191      +29     
  Lines        5509     7116    +1607     
  Branches     1027     1364     +337     
==========================================
+ Hits         1750     2001     +251     
- Misses       3552     4900    +1348     
- Partials      207      215       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@AWSHurneyt AWSHurneyt left a comment

Choose a reason for hiding this comment

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

Approved assuming CI passes.

@amsiglan
Copy link
Collaborator Author

Tests passed:

image

@amsiglan amsiglan merged commit d3548eb into opensearch-project:main Aug 10, 2024
7 of 10 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 10, 2024
* updated vega imports; fixed get correlated findings

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* fixed correlation alerts filtering; improved list iocs pagination

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated help text for data source selection

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* fixed active condition for threat intel source

Signed-off-by: Amardeepsingh Siglani <[email protected]>

---------

Signed-off-by: Amardeepsingh Siglani <[email protected]>
(cherry picked from commit d3548eb)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 10, 2024
* updated vega imports; fixed get correlated findings

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* fixed correlation alerts filtering; improved list iocs pagination

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated help text for data source selection

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* fixed active condition for threat intel source

Signed-off-by: Amardeepsingh Siglani <[email protected]>

---------

Signed-off-by: Amardeepsingh Siglani <[email protected]>
(cherry picked from commit d3548eb)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 10, 2024
* updated vega imports; fixed get correlated findings

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* fixed correlation alerts filtering; improved list iocs pagination

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated help text for data source selection

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* fixed active condition for threat intel source

Signed-off-by: Amardeepsingh Siglani <[email protected]>

---------

Signed-off-by: Amardeepsingh Siglani <[email protected]>
(cherry picked from commit d3548eb)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
amsiglan pushed a commit that referenced this pull request Aug 10, 2024
* updated vega imports; fixed get correlated findings



* fixed correlation alerts filtering; improved list iocs pagination



* updated help text for data source selection



* fixed active condition for threat intel source



---------


(cherry picked from commit d3548eb)

Signed-off-by: Amardeepsingh Siglani <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
amsiglan added a commit that referenced this pull request Aug 10, 2024
* Fix UI issues (#1107)

* updated vega imports; fixed get correlated findings

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* fixed correlation alerts filtering; improved list iocs pagination

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated help text for data source selection

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* fixed active condition for threat intel source

Signed-off-by: Amardeepsingh Siglani <[email protected]>

---------

Signed-off-by: Amardeepsingh Siglani <[email protected]>
(cherry picked from commit d3548eb)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* updated snapshots

Signed-off-by: Amardeepsingh Siglani <[email protected]>

---------

Signed-off-by: Amardeepsingh Siglani <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Amardeepsingh Siglani <[email protected]>
amsiglan pushed a commit that referenced this pull request Aug 10, 2024
* updated vega imports; fixed get correlated findings



* fixed correlation alerts filtering; improved list iocs pagination



* updated help text for data source selection



* fixed active condition for threat intel source



---------


(cherry picked from commit d3548eb)

Signed-off-by: Amardeepsingh Siglani <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants