Skip to content

[sentinel_one] Improve UI Layout to Input Section - #15862

Merged
mohitjha-elastic merged 3 commits into
elastic:mainfrom
mohitjha-elastic:sentinel_one-improve-ui-layout
Nov 11, 2025
Merged

[sentinel_one] Improve UI Layout to Input Section#15862
mohitjha-elastic merged 3 commits into
elastic:mainfrom
mohitjha-elastic:sentinel_one-improve-ui-layout

Conversation

@mohitjha-elastic

@mohitjha-elastic mohitjha-elastic commented Nov 4, 2025

Copy link
Copy Markdown
Contributor

Proposed commit message

sentinel_one: Improve UI layout to the input section.

Improved the UI layout of the input section for better clarity and alignment.  
Updated input titles to include relevant data stream names.  
Removed the CEL and HTTPJSON labels to simplify the interface.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

How to test this PR locally

  • Clone integrations repo.
  • Install elastic package locally.
  • Start elastic stack using elastic-package.
  • Move to integrations/packages/sentinel_one directory.
  • Run the following command to run tests.

elastic-package test -v

Related Issue

Screenshots

Before:

before_s1

After:
after_s1

@mohitjha-elastic mohitjha-elastic self-assigned this Nov 4, 2025
@mohitjha-elastic
mohitjha-elastic requested a review from a team as a code owner November 4, 2025 16:49
@mohitjha-elastic mohitjha-elastic added enhancement New feature or request Integration:sentinel_one SentinelOne Category: Integration quality Category: Quality used for SI planning Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] labels Nov 4, 2025
@elasticmachine

Copy link
Copy Markdown

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@kcreddy kcreddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you update manifest and add changelog entry?

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @mohitjha-elastic

@andrewkroh andrewkroh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Since this claims to affect the UI layout, can you please post a before and after screenshot into the PR description in this section:

## Screenshots
<!-- Optional
Add here screenshots presenting:
- Kibana UI forms presenting configuration options exposed by the integration
- dashboards with collected metrics or logs
-->

I'm curious to see what title/description changes do to the layout. Thanks.

@kcreddy kcreddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@mohitjha-elastic, LGTM for my comments. Please address Andrew's comments as well so I can approve.

@mohitjha-elastic

Copy link
Copy Markdown
Contributor Author

Updated input titles to include relevant data stream names.
Removed the CEL and HTTPJSON labels to simplify the interface.

@andrewkroh
This change updates the input titles to include relevant data stream names by removing the CEL and HTTPJSON labels to simplify the interface for the user.
Added the screenshots in the PR description.

@andrewkroh andrewkroh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think calling this a "layout" (arrangement) change might be overstating the effect, but LGTM.

@mohitjha-elastic
mohitjha-elastic merged commit 7bfc12d into elastic:main Nov 11, 2025
7 checks passed
@mohitjha-elastic
mohitjha-elastic deleted the sentinel_one-improve-ui-layout branch November 11, 2025 07:12
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package sentinel_one - 1.42.0 containing this change is available at https://epr.elastic.co/package/sentinel_one/1.42.0/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Category: Integration quality Category: Quality used for SI planning enhancement New feature or request Integration:sentinel_one SentinelOne Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

4 participants