-
-
Notifications
You must be signed in to change notification settings - Fork 159
One hour added to time when selecting date in DST with a fixed timezone. #1059
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
Comments
@aostijn is this bug solved for you now? @Jasenkoo I am still having this exact issue for timezone |
Yes, it's solved for me. The problem you're facing is probably another one, which I ran into as well a couple of days ago. It doesn't seem to be a DST/non-DST problem (because Mar 9 is still outside of DST). I've created a new issue: #1091. Take a look and see if this is actually the issue you're facing. Cheers :) |
This should have been marked as a breaking change tbh, as this fix caused our specs to fail when upgrading from v9 to v11. The emitted Date value is now 1 hour less, and this can result in a whole day difference when converting between timezones. Don't get me wrong, good to have the bug fixed if it was an actual issue, but a bit of a head scratcher to figure out why the output suddenly changed by an hour. |
Actually I think this change introduced a bug, I will raise a new ticket for it. |
Hi! First of all, thanks a lot for the work on this tool, it's great! I ran into an issue that somewhat resembles #991 but doesn't seem to be entirely the same (#991 occurred in 9.0.3 and should be fixed with 10.0.0 but for my problem, it's the opposite), so I created this bug report.
Describe the bug
I provide a fixed timezone with the
timezone
string prop (in my case: 'Europe/Brussels'). At the time of writing (December), we're in winter time, so outside of DST. When I choose a date and time outside of DST (for example in January), everything works as expected. But when I choose a date and time inside of DST (for example in June), when I confirm, one hour is added to the time I chose.Some notes that might be useful:
To Reproduce
Steps to reproduce the behavior:
Expected behavior
The time should be the same as selected.
Screenshots

Desktop & mobile (please complete the following information):
The text was updated successfully, but these errors were encountered: