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.

Sign up for GitHub

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

Jump to bottom

[Cloud Security] Change update dataview #183789

Merged
merged 6 commits into from
May 22, 2024
Merged

[Cloud Security] Change update dataview #183789

merged 6 commits into from
May 22, 2024

Conversation

animehart
Copy link
Contributor

@animehart animehart commented May 17, 2024 •
edited

Summary

Removing Update logic for Findings table from Client Side as well as overriding table column name

@animehart
Copy link
Contributor Author

/ci

@animehart
Copy link
Contributor Author

/ci

@animehart
Copy link
Contributor Author

/ci

@animehart
Copy link
Contributor Author

/ci

@animehart animehart linked an issue May 22, 2024 that may be closed by this pull request
[Cloud Security] Remove DataView labels update logic and override table columns instead #172615
Closed
5 tasks
@animehart animehart changed the title Change update dataview [Cloud Security] Change update dataview May 22, 2024
@animehart animehart marked this pull request as ready for review May 22, 2024 09:05
@animehart animehart requested a review from a team as a code owner May 22, 2024 09:05
@animehart animehart added v8.15.0 release_note:skip Skip the PR/issue when compiling release notes Team:Cloud Security Cloud Security team related labels May 22, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-cloud-security-posture (Team:Cloud Security)

@kibana-ci
Copy link
Collaborator

💚 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
cloudSecurityPosture 448.7KB 446.4KB -2.3KB

History

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

opauloh
opauloh approved these changes May 22, 2024
View reviewed changes
Copy link
Contributor

@opauloh opauloh left a comment •
edited

Choose a reason for hiding this comment

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

Tested it locally - LGTM

Worth noting that after this change is merged the tables won't have the custom naming by default anymore as the UnifiedDataTable does not support custom column naming, only DataView naming. Please create a follow-up ticket to add custom column naming support (Independent from the DataView) to the UnifiedDataTable component.

@animehart animehart merged commit 9cb7940 into elastic:main May 22, 2024
18 checks passed
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label May 22, 2024
@animehart animehart mentioned this pull request May 23, 2024
Open
2 tasks
rshen91 pushed a commit to rshen91/kibana that referenced this pull request May 30, 2024
@animehart @rshen91
[Cloud Security] Change update dataview (elastic#183789)
a203d32 
## Summary

Removing Update logic for Findings table from Client Side as well as
overriding table column name
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@opauloh opauloh opauloh approved these changes

Assignees
No one assigned
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Cloud Security Cloud Security team related v8.15.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Cloud Security] Remove DataView labels update logic and override table columns instead
5 participants
@animehart @elasticmachine @kibana-ci @opauloh @kibanamachine