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

Add Time Zones #764

Closed
Starpaul20 opened this issue Jun 20, 2014 · 6 comments
Closed

Add Time Zones #764

Starpaul20 opened this issue Jun 20, 2014 · 6 comments
Assignees
Labels
b:1.8 Branch: 1.8.x s:resolved Status: Resolved. Solution implemented or scheduled t:enhancement Type: Enhancement. Contains minor improvements
Milestone

Comments

@Starpaul20
Copy link
Member

Several official time zones, such as UTC-9:30, UTC+05:45, UTC+06:30, UTC+10:30, UTC+11:30, UTC+12:45, UTC+13:00 and UTC+14:00 are missing and should be added.

Also need to fix a bug where :45 timezone appear as :75 in the time zone drop down.

@Starpaul20 Starpaul20 added this to the 1.8 Beta 2 milestone Jun 20, 2014
@Starpaul20 Starpaul20 self-assigned this Jun 20, 2014
Starpaul20 added a commit that referenced this issue Jun 20, 2014
@Starpaul20
Copy link
Member Author

Finished

Starpaul20 added a commit that referenced this issue Jun 20, 2014
Added setting upgrade routine and updated time zone city list
@DiogoParrinha
Copy link
Contributor

I can confirm this to be working.

@Destroy666x
Copy link
Contributor

$l['timezone_gmt_200'] = "(GMT +2:00) Athens, Istanbul, Cairo, Jerusalem, South Africa, Warsaw";

Warsaw (like whole Poland) has CET, so it's GMT + 1 without summer time like in Berlin, Copenhagen etc. We never have the same time as Greece. This need to be corrected. Not sure if the rest is correct, but probably it needs to be checked.

BTW, is there any reason why the short GMT list is used UCP options and the long list in ACP edit user? I think both should be long.

@DiogoParrinha
Copy link
Contributor

@PaulBender

@DiogoParrinha DiogoParrinha reopened this Jun 24, 2014
Starpaul20 added a commit that referenced this issue Jun 24, 2014
Updated time zone list
@Starpaul20
Copy link
Member Author

Okay, I've updated the entire list. All cities/countries listed should be correct.

@DiogoParrinha
Copy link
Contributor

Awesome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
b:1.8 Branch: 1.8.x s:resolved Status: Resolved. Solution implemented or scheduled t:enhancement Type: Enhancement. Contains minor improvements
Projects
None yet
Development

No branches or pull requests

3 participants