Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Uruguayan/Montevideo time zone (UYT/UYST) is missing in Rails #9628

Closed
GSI opened this Issue Mar 9, 2013 · 5 comments

Comments

Projects
None yet
4 participants

GSI commented Mar 9, 2013

It surprises me, but it seems UYT, and it's DST variant (UYST) are missing.

UYT is GMT-3. There are other time zones with GMT-3, but they differ according to their DST's:

  • Brasilia (BRT) is outside DST from 17th Feb 2013 - 20th Oct 2013
  • Buenos Aires (ART) has no DST (respect! :))
  • Greenland (WGT) is outside DST from 26. Oct 2013 - 30th Mar 2013
  • Uruguay (UYU) is outside DST: 10th Mar 2013 - 6th Oct 2013

In order to allow for correct time zone handling, Uruguay should be included in the list of zones.

Details regarding the time zone:
http://www.timeanddate.com/worldclock/city.html?n=163

Owner

pixeltrix commented Mar 9, 2013

It is there under 'America/Montevideo', which is the preferred method of specifying timezones as required by ISO8601, e.g:

>> Time.zone = 'America/Montevideo'
=> "America/Montevideo"
>> Time.current
=> Sat, 09 Mar 2013 15:34:01 UYST -02:00

ActiveSupport::TimeZone has a list of aliases that is used to populate the timezone select helper amongst other things and I think we should probably add an alias there for 'Montevideo' => 'America/Montevideo' since Rails is so popular in Uruguay. 😄

Do you want to make a PR or should I go ahead and make the change?

@GSI GSI pushed a commit to GSI/rails that referenced this issue Mar 9, 2013

Gernot included "Montevideo" in list of time zone aliases (closes Issue #9628) 0bfaa2c

GSI commented Mar 9, 2013

It sure is :)

When I looked for the supported zones I queried the class with the aliases (via ActiveSupport::TimeZone.all).

How would I have gotten to an actually complete list?

Owner

pixeltrix commented Mar 10, 2013

@GSI in the console you could do this:

>> TZInfo::Timezone.all.each{ |tz| puts tz.name }; nil

which prints out approximately 580 names

Owner

spastorino commented Mar 10, 2013

ZOMG ❤️ ❤️ ❤️

Uruguay

GSI commented Mar 10, 2013

Thanks for the hint (@pixeltrix) and the picture (@spastorino) 😆

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