CUMULUS-3892: Update the home page metrics overview section

[Nnaga1]

Summary: Summary of changes

Addresses CUMULUS-3892: Dashboard - Home Page Metrics Overview Section Update

Changes

• Code: Update timeframe to the past 24 hours
• UI: Add a label to inform user of the timeframe

PR Checklist

• Update CHANGELOG
• Unit tests
• Adhoc testing
• Integration tests

[jjmccoy]

@Nnaga1 This looks good. I believe there is a cypress test for the home page: main_page_spec.js. That may need updating for the timeframe update change. If not, just lmk.

[jjmccoy]

@Nnaga1

FYI: So if the user goes to another page and comes back to the home page, the stats from that page temporarily bring those over to the home page and it refreshes. In this video example, it is Rules (for ingest) that has 34 currently. For this, I would suggest creating a ticket for the issue because it is outside the scope of your ticket but may need to be addressed so that users are not confused or think the information is misleading.

Action needed: Create new ticket

Screen.Recording.2024-10-16.at.12.14.05.PM.mov

[Nnaga1]

@Nnaga1

FYI: So if the user goes to another page and comes back to the home page, the stats from that page temporarily bring those over to the home page and it refreshes. In this video example, it is Rules (for ingest) that has 34 currently. For this, I would suggest creating a ticket for the issue because it is outside the scope of your ticket but may need to be addressed so that users are not confused or think the information is misleading.

Action needed: Create new ticket

Screen.Recording.2024-10-16.at.12.14.05.PM.mov

Dang I was unaware of that, thanks for lmk, I'll try to see if I can figure it out within this ticket, if not I'll create another, thanks 😄

[Nnaga1]

I did some additional testing on the issue you mentioned @jjmccoy, it seems like it does this not only for rules/executions for the overview only, if you go from home -> granules, the inverse will happen. It will show all 0's for the statuses before the actual counts. Same with if you go from a different page back to the home page, the granule errors will flash from being a list of what it was before to what it should be (i.e. it shows an old list for one second before changing to what it actually should show). It seems like in some cases/calls this behavior occurs. I think I will create a ticket detailing this, but I really dont think it is worth prioritizing at all probably, since it only lasts for a second and is somewhat persistent across the dashboard. I'm not going to consider that in the scope of this ticket. I think this ticket: #1153, should be reviewed before this is approved in order to clear the CI issues.

[jjmccoy]

I did some additional testing on the issue you mentioned @jjmccoy, it seems like it does this not only for rules/executions for the overview only, if you go from home -> granules, the inverse will happen. It will show all 0's for the statuses before the actual counts. Same with if you go from a different page back to the home page, the granule errors will flash from being a list of what it was before to what it should be (i.e. it shows an old list for one second before changing to what it actually should show). It seems like in some cases/calls this behavior occurs. I think I will create a ticket detailing this, but I really dont think it is worth prioritizing at all probably, since it only lasts for a second and is somewhat persistent across the dashboard. I'm not going to consider that in the scope of this ticket. I think this ticket: #1153, should be reviewed before this is approved in order to clear the CI issues.

Yeah, I figured it was out of scope and a new ticket needs to be created for the issue.

[jjmccoy]

@Nnaga1 This looks good. I believe there is a cypress test for the home page: main_page_spec.js. That may need updating for the timeframe update change. If not, just lmk.

I can approve it if this Cypress test doesn't need to be updated.

[jjmccoy]

@Nnaga1 Thanks for testing it. LGTG!