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 for timezones with minute Offsets >0 #1599
Fix for timezones with minute Offsets >0 #1599
Conversation
…ible by 60. For Indian Timezone +05:30 or 330 minutes the output becomes +5.5:30 but as per ISO 8601 standard it should be +05:30. hourOffset should be "05" and minute offset should be "30" but currently hourOffset is set to "5.5"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah, right! I completely missed the non-standard time zones. Thank you for fixing this!
Still not merged? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's good to go. Thank you a lot! I'll merge and ship it soon.
I found this PR only after also implementing the fix. If you want a test that actually tests the output to go with this, feel free to use: mtkopone@c0f4396 |
@mtkopone that's perfect, thank you a lot for this! |
I was overwhelmed lately, thank you for your patience! The fix was released with |
For example Indian Standard Timezone IST (https://en.wikipedia.org/wiki/Indian_Standard_Time) +05:30 or 330 minutes the output becomes +5.5:30 but as per ISO 8601 standard it should be +05:30. hourOffset should be "05" and minute offset should be "30" but currently hourOffset is set to "5.5"