-
Notifications
You must be signed in to change notification settings - Fork 554
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
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
Time and date picker adjusts time based on timezone #2558
Comments
It's probably because in Edit: Had to also edit flatpickr format on line 135 to |
The offset is intended to improve the user experience. Dates should always be stored in UTC in the database, but you don't want your users to have to do mental gymnastics with timezones when using date and time pickers in the CMS. Are you reporting a bug with this implementation, or is it more that you were looking to disable that behavior? |
To be honest, at the point when I opened this issue I was completely oblivious to the fact that it was good practice to store UTC dates in the DB - answering your question - it's not a bug if you followed best practices, so we might as well close this. But still it would be nice to be able to store and display datetime values independently from timezones. Do you have an idea if it would be possible currently? |
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
Description
time_picker
field would adjust the value displayed in the field based on OS/browser timezone ie. will not display the value stored in the DB, but treats that value as if it was UTC and the displayed value was local. Value in store is correct so it should be something related to frontend/flatpickr.js.Edit:
date_picker
with time is also affected.Steps to reproduce
Set your OS timezone to something other than UTC and compare DB stored value and the field.
Expected result
No timezone adjustment
Actual result
Value in field offset by timezone
Versions
Twill version: 2.3.0
Laravel version: 10
The text was updated successfully, but these errors were encountered: