Fix asset uptime for phases of day #5894
Merged
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.
WHAT
🤖 Generated by Copilot at 23bbbe2
Improved the UI and functionality of the
Uptime
component for assets. Fixed bugs, refactored code, and optimized data filtering and caching.Proposed Changes
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
HOW
🤖 Generated by Copilot at 23bbbe2
STATUS_COLORS
object to use more descriptive and consistent keys for the asset status values (link)statusToCarryOver
variable to use the last record of the previous day instead of the first record (link)recordsInPeriodCache
object to store the filtered records for each hour of the day and improve performance (link)recordsInPeriod
variable from therecordsInPeriodCache
object if it exists, or from the original filtering logic otherwise (link)previousLatestRecord
variable to store the last record of the previous hour and use it for checking the status transition at the end of the hour (link)previousLatestRecord
variable and thestatus
property of the record, and change the default value to "Not Monitored" (link)