What about links? about timezone

[jhvst]

Hi and thanks for submitting the issue!

I generated the tables from CSV files downloaded from a third party website, which seemingly did not include the links. I did not have any reference library to build from, so the absence never came up.

Testing the underlying Offset code in Go Playground shows that the link locations can be parsed with Go's time package. Therefore, I agree that the links should be added.

I wouldn't mind a pull request, but I'll try to be productive enough to add the missing locations by myself during this week.

[aviflax]

Cool. I don’t think I’ll be able to submit a contribution, sorry. But thanks!

[jhvst]

Hi again,

It looks like there would be 88 location additions, including the following for US:

US/Alaska
US/Aleutian
US/Arizona
US/Central
US/East-Indiana
US/Eastern
US/Hawaii
US/Indiana-Starke
US/Michigan
US/Mountain
US/Pacific
US/Pacific-New
US/Samoa

Do you notice anything missing?

I did remove few backwards compatible entries such as these, since most of them aren't really locations, which makes time inaccurate. The updated location list array will be made from the following file: https://github.com/9uuso/timezone/blob/dev/2015e/loc

Can you check are the backwards compatible timezones listed in your frontend library? If they are, I have to figure out how to deal with the backward compatible zones.

If these changes satisfy you, I'll generate the Go code and merge the changes.

[aviflax]

Honestly I think rather than try to base the list on any given client behavior, we should stick with the canonical IANA list. I’m hoping that the Wikipedia page is an accurate representation of that list.

I extracted a list of all the values from that page. I count 538 values in that list, whereas the file you linked in your dev branch has only 504.

My preference would be for the lib to include all 538 values listed in the Wikipedia page.

[jhvst]

I downloaded the latest package from IANA (version 2015e -- Wikipedia has 2014b) and updated the locations to include timezone links. The new array is 584 elements long and includes the timezones that you were missing.

[aviflax]

Excellent, thanks!!