Add timestamp formatting to chart tooltips #1152
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
|
Thanks @Rudra-Garg! Could you please sign-off your commits, so the DCO check passes? You can find instructions by clicking on the Details link next to the failing DCO check. |
Previously chart tooltips only showed dates without time information, making it difficult to correlate chart events with specific times. This change adds proper timestamp formatting (including HH:mm:ss) to tooltips across all chart components. The changes include: - Added timestamp formatting in chart tooltips using common.formatTimestamp() - Applied consistent formatting across all chart components - Maintained existing date formatting on x-axis labels This addresses issue DependencyTrack#1142 which requested adding timestamp information to make it easier to investigate events shown in charts. Signed-off-by: Rudra Garg <rudra.garg22b@iiitg.ac.in>
1069152 to
c9f67c4
Compare
Contributor
Author
|
This pull request can be merged now I think |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Previously chart tooltips only showed dates without time information, making it difficult to correlate chart events with specific times. This change adds proper timestamp formatting (including HH:mm:ss) to tooltips across all chart components.
The changes include:
This addresses issue #1142 which requested adding timestamp information to make it easier to investigate events shown in charts.
Description
Added timestamp (HH:mm:ss) formatting to all chart tooltips to help users better
correlate visualization data points with system events. This enhancement makes it
easier to investigate peaks, troughs, and step changes by matching them with logs
and CI/CD pipeline activities.
Addressed Issue
Resolves #1142
Additional Details
The changes include:
All dashboard chart and widget components have been updated to ensure consistent
timestamp display. The implementation uses the existing formatTimestamp() utility
to maintain consistency across the application.
Before:
After:
Checklist