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

Fix event selection for daylight saving changes #782

Merged
merged 1 commit into from
Apr 18, 2022

Conversation

lucc
Copy link

@lucc lucc commented Apr 4, 2022

On days where daylight saving is toggled the day does not last exactly
86400000 ms. In the timezone Europe/Berlin for example daylight saving
time was toggled on 2022-03-27 at 02:00 hours. This results in
2022-03-27T00:00:00 + 86400000 ms = 2022-03-28T01:00:00. Therefore
events where displayed on the wrong day.

On days where daylight saving is toggled the day does not last exactly
86400000 ms.  In the timezone Europe/Berlin for example daylight saving
time was toggled on 2022-03-27 at 02:00 hours.  This results in
2022-03-27T00:00:00 + 86400000 ms = 2022-03-28T01:00:00.  Therefore
events where displayed on the wrong day.
@Serhioromano Serhioromano merged commit 3422459 into Serhioromano:master Apr 18, 2022
@lucc lucc deleted the daylight-saving branch April 19, 2022 08:26
@lucc
Copy link
Author

lucc commented Apr 19, 2022

@Serhioromano thank you for merging. Do you mind tagging a release? The last one is quite old and having a recent release could help new people coming to the repository to not ask #778 again.

Also it would help me to bundle the new version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants