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][Timeline] Remove BrowserField type #188923

Merged
merged 8 commits into from
Aug 2, 2024

Conversation

lgestc
Copy link
Contributor

@lgestc lgestc commented Jul 23, 2024

Summary

This PR removes BrowserField type, as it is now replaced with data plugins' FieldSpec.

@lgestc lgestc requested review from a team as code owners July 23, 2024 11:42
@lgestc lgestc requested a review from dhurley14 July 23, 2024 11:42
@lgestc lgestc added release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting Team:Threat Hunting:Investigations Security Solution Investigations Team labels Jul 23, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-threat-hunting-investigations (Team:Threat Hunting:Investigations)

Copy link
Contributor

@dhurley14 dhurley14 left a comment

Choose a reason for hiding this comment

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

👍

@lgestc lgestc force-pushed the remove_browser_field_type branch from b689d5e to 915ede1 Compare July 25, 2024 11:02
@@ -25,7 +26,7 @@ export const allowSorting = ({
browserField,
fieldName,
}: {
browserField: Partial<BrowserField> | undefined;
browserField: Partial<FieldSpec> | undefined;
Copy link
Contributor

@michaelolo24 michaelolo24 Jul 26, 2024

Choose a reason for hiding this comment

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

❤️

Copy link
Contributor

@kqualters-elastic kqualters-elastic left a comment

Choose a reason for hiding this comment

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

lgtm 👍

@lgestc lgestc enabled auto-merge (squash) July 27, 2024 20:16
@lgestc lgestc disabled auto-merge August 1, 2024 07:57
@lgestc lgestc force-pushed the remove_browser_field_type branch from 842905f to 30a5ad3 Compare August 1, 2024 09:27
@elastic elastic deleted a comment from kibana-ci Aug 1, 2024
@lgestc lgestc force-pushed the remove_browser_field_type branch from 30a5ad3 to 9131eb0 Compare August 1, 2024 10:46
@elastic elastic deleted a comment from kibana-ci Aug 1, 2024
@elastic elastic deleted a comment from kibana-ci Aug 1, 2024
@elastic elastic deleted a comment from kibana-ci Aug 1, 2024
@lgestc lgestc enabled auto-merge (squash) August 2, 2024 07:36
@elastic elastic deleted a comment from kibana-ci Aug 2, 2024
@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
timelines 184 183 -1
Unknown metric groups

API count

id before after diff
timelines 228 227 -1

References to deprecated APIs

id before after diff
@kbn/securitysolution-data-table 21 15 -6
securitySolution 511 473 -38
total -44

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

@lgestc lgestc merged commit efb71fa into elastic:main Aug 2, 2024
42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting:Investigations Security Solution Investigations Team v8.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants