-
Notifications
You must be signed in to change notification settings - Fork 174
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
Messed up timezones after PR #417 #422
Comments
Thanks for raising this one. Looks like the graph update mechanism was missed on this one for the memory and CPU stats, as they slightly vary from the queue implementation. I've opened a PR to address the wrong formatting after updates: #423 If custom cards want to format their dates, they will need to manually make this happen. There is a global I'm unable to replicate the issue with the queues card date. If you can provide the steps to reproduce the problem I will reopen the issue and dive into it. |
There are no specific steps to reproduce. It just works that way. I can describe the server and client configuration. Server:
Client:
The tooltip issue occurs not only on the Queues card, but on other cards as well. |
Hmm. Can you please try upgrading to |
Appreciate you checking this again. I've opened a PR to fix this: #425 |
Pulse Version
1.2.6
Laravel Version
11.33.2
PHP Version
8.3.13
Livewire Version
3.5.12
Database Driver & Version
No response
Description
After #417 timezones are messed up much more than before.
It's 16:56 CET here now.
CPU and memory graphs show UTC time (so 1 hour back):
Queues graph shows CET time (correct):
However queues card title tooltip shows some mystery timezone (1 hour forward, definitely not CET):
Custom cards just use UTC. I'm guessing they should be manually modified by their authors, but do the authors know about it?
Steps To Reproduce
Just observe the Pulse cards.
The text was updated successfully, but these errors were encountered: