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

Incorrect timezone considerations #4785

Closed
th3fallen opened this issue Sep 21, 2018 · 2 comments
Closed

Incorrect timezone considerations #4785

th3fallen opened this issue Sep 21, 2018 · 2 comments

Comments

@th3fallen
Copy link

th3fallen commented Sep 21, 2018

Description of the Issue and Steps to Reproduce:
Moment is configured in America/Asuncion timezone (-4) calculations are wrong when adding days.

moment example https://runkit.com/embed/1r62d83amq7x
yet luxon handles this correctly
https://runkit.com/embed/t49achvensqf

Please include the values of all variables used.

Environment:

Examples: Chrome 49 on OSX, Internet Explorer 10 on Windows 7, Node.JS 4.4.4 on Ubuntu 16.0.4
Chrome Latest, OSX, node LTS, React
Both the browser and the OS are important to us, particularly if you have an unsual environment like an IOT application.

Other information that may be helpful:

  • The time zone setting of the machine the code is running on CST or EST
  • The time and date at which the code was run America/Asuncion
  • Other libraries in use (TypeScript, Immutable.js, etc) moment-timezone in this case immutable.js are in the codebase but not relevant to this

If you are reporting an issue, please run the following code in the environment you are using and include the output:

America/Asuncion
VM25952:13 Fri Sep 21 2018 13:34:12 GMT-0400 (Eastern Daylight Time)
VM25952:14 9/21/2018, 1:34:12 PM
VM25952:15 240
VM25952:16 Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/69.0.3497.100 Safari/537.36
VM25952:17 2.22.1

Ensure your issue is isolated to moment. Issues involving third party tools will be closed unless submitted by the tool's author/maintainer.

@th3fallen th3fallen changed the title Incorrect timezone considerations Incorrect timezone considerations with DST Sep 21, 2018
@th3fallen th3fallen changed the title Incorrect timezone considerations with DST Incorrect timezone considerations Sep 21, 2018
@marwahaha
Copy link
Member

We need more information here to make any changes.

@marwahaha
Copy link
Member

This is interesting. I'm going to close as duplicate of #4743

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

No branches or pull requests

2 participants