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

[8.2.1] [Session view] Fix timestamps format #130568

Merged
merged 2 commits into from
Apr 25, 2022

Conversation

opauloh
Copy link
Contributor

@opauloh opauloh commented Apr 19, 2022

it fixes #130563

Summary

Update Timestamp format to use a 00 to 23 Hour range, so users can distinguish between an AM or PM Time.

Screenshots

Process on Session View (15:42:31):
image

Same process on Discover (15:42:31):
image

@opauloh opauloh added release_note:skip Skip the PR/issue when compiling release notes auto-backport Deprecated - use backport:version if exact versions are needed Team: AWP: Platform Adaptive Workload Protection Platform team from Security Solution v8.3.0 v8.2.1 labels Apr 19, 2022
@opauloh opauloh requested a review from a team April 19, 2022 14:47
@opauloh opauloh self-assigned this Apr 19, 2022
Copy link

@zizhouW zizhouW left a comment

Choose a reason for hiding this comment

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

LGTM!

@opauloh opauloh added the v8.2.0 label Apr 19, 2022
@@ -13,7 +13,7 @@ export const PROCESS_EVENTS_INDEX = 'logs-endpoint.events.process-*';
export const PREVIEW_ALERTS_INDEX = '.preview.alerts-security.alerts-default';
export const ENTRY_SESSION_ENTITY_ID_PROPERTY = 'process.entry_leader.entity_id';
export const ALERT_UUID_PROPERTY = 'kibana.alert.uuid';
export const KIBANA_DATE_FORMAT = 'MMM DD, YYYY @ hh:mm:ss.SSS';
export const KIBANA_DATE_FORMAT = 'MMM DD, YYYY @ HH:mm:ss.SSS';
Copy link
Contributor

Choose a reason for hiding this comment

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

I think eventually this should come from the user configurable global kibana setting, example can be found here https://github.com/elastic/kibana/blob/main/x-pack/plugins/security_solution/public/common/lib/kibana/hooks.ts#L27

Copy link
Contributor Author

@opauloh opauloh Apr 19, 2022

Choose a reason for hiding this comment

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

good point, I will add that in our backlog for us to improve later, thanks!

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

cc @opauloh

@opauloh opauloh merged commit dd5b807 into elastic:main Apr 25, 2022
@opauloh opauloh deleted the fix-session-view-timestamps branch April 25, 2022 16:11
@opauloh opauloh added Team: AWP: Visualization AWP team that does most fullstack work in kibana and removed Team: AWP: Platform Adaptive Workload Protection Platform team from Security Solution labels Apr 25, 2022
kibanamachine pushed a commit that referenced this pull request Apr 25, 2022
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.2

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Apr 25, 2022
(cherry picked from commit dd5b807)

Co-authored-by: Paulo Henrique <[email protected]>
kertal pushed a commit to kertal/kibana that referenced this pull request May 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes Team: AWP: Visualization AWP team that does most fullstack work in kibana v8.2.0 v8.2.1 v8.3.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Session View] Timestamps are formatted incorrectly
5 participants