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 chart time display when the system timezone has a positive offset from UTC #1035

Merged
merged 1 commit into from
Dec 10, 2019

Conversation

jpwhite4
Copy link
Member

Fix issue with the chart tooltip displaying the wrong date when
the system timezone is a negative offset from UTC. That is countries
located east of the prime meridian.

See https://help.xdmod.org/a/tickets/14165

@jpwhite4 jpwhite4 added the bug Bugfixes label Sep 11, 2019
@jpwhite4 jpwhite4 added this to the 9.0.0 milestone Sep 11, 2019
Fix issue with the chart tooltip displaying the wrong date when
the system timezone is a negative offset from UTC. That is countries
located east of the prime meridian.

See https://help.xdmod.org/a/tickets/14165
@plessbd
Copy link
Contributor

plessbd commented Sep 11, 2019

timezones to the east of UTC have Positive offsets https://en.wikipedia.org/wiki/List_of_UTC_time_offsets

@plessbd plessbd changed the title Fix chart time display when the system timezone is a negative offset from UTC Fix chart time display when the system timezone has a positive offset from UTC Sep 18, 2019
@plessbd plessbd changed the base branch from xdmod8.5 to xdmod8.6 October 1, 2019 13:09
@plessbd plessbd modified the milestones: 9.0.0, 8.6.0 Oct 1, 2019
@jpwhite4 jpwhite4 modified the milestones: 8.6.0, 9.0.0 Oct 23, 2019
@jtpalmer jtpalmer changed the base branch from xdmod8.6 to xdmod9.0 October 23, 2019 16:32
@jpwhite4 jpwhite4 merged commit 085fc5d into ubccr:xdmod9.0 Dec 10, 2019
@jpwhite4 jpwhite4 deleted the chart_timezone branch December 10, 2019 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bugfixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants