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

Message status, daily chart, & All Activity chart - stats should be the same? #2204

Open
Tracked by #2171
heyitsmebev opened this issue Dec 11, 2024 · 4 comments · May be fixed by #2314
Open
Tracked by #2171

Message status, daily chart, & All Activity chart - stats should be the same? #2204

heyitsmebev opened this issue Dec 11, 2024 · 4 comments · May be fixed by #2314

Comments

@heyitsmebev
Copy link
Contributor

heyitsmebev commented Dec 11, 2024

Stats throughout the site may not be accurate. They are being pulled from different endpoints. Are the stats consistent with UTC vs local time?

image image image
@heyitsmebev heyitsmebev changed the title Message status, daily chart, and All Activity chart, stats should be the same? Message status, daily chart, & All Activity chart - stats should be the same? Dec 11, 2024
@heyitsmebev
Copy link
Contributor Author

After talking to Carlo, some data are accounting for timezone/local time and others are not. For example, the dashboard chart is probably getting UTC time but not displaying in local time.

@heyitsmebev
Copy link
Contributor Author

Note: the chart takes a bit longer to update than on the Jobs page because of how the delivered/failed are being updated with the endpoint we are using

@alexjanousekGSA
Copy link
Contributor

So is this a bug or a feature? Is there something important to fix to make them sync up quicker?

@ccostino
Copy link
Contributor

ccostino commented Feb 7, 2025

I followed up in the API PR that was opened for this: GSA/notifications-api#1547 (comment)

Quick summary: the timezone conversions and handling should happen here on the admin side, because the API only knows and cares about UTC when dealing with dates and times.

As far as the syncing part, that I'm less sure about but if I remember correctly that's also more on the admin side. @terrazoon do you have any other insight for what may or may not be needed on the API side?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment