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

Zaporizhzhia name update #11

Merged
merged 4 commits into from
Jan 13, 2024
Merged

Conversation

Ilyahas
Copy link
Contributor

@Ilyahas Ilyahas commented Mar 10, 2023

https://en.wikipedia.org/wiki/Zaporizhzhia
City in Ukraine. I saw you did changes for Kyiv, here is the same situation.

Ilyahas added 2 commits March 10, 2023 14:09
https://en.wikipedia.org/wiki/Zaporizhzhia 
City in Ukraine, I saw you did changes for Kyiv, here is the same situation.
@omsrivastava
Copy link
Owner

Hi @Ilyahas, thanks for the update.
Just want to confirm. Is there any other city of Ukraine whose name needs to be updated? Then we can update it in the same version publishing.

@Ilyahas
Copy link
Contributor Author

Ilyahas commented Mar 10, 2023

Just found: "Odessa" has to be "Odesa"

https://en.wikipedia.org/wiki/Odesa

https://github.com/omsrivastava/timezones-list/blob/master/src/timezones.json#L1601

@Ilyahas
Copy link
Contributor Author

Ilyahas commented Mar 21, 2023

@omsrivastava can you update version and let's merge it :)

Copy link

@VitaliiLazebnyi VitaliiLazebnyi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR still needs updating of versions just like it was done here: https://github.com/omsrivastava/timezones-list/pull/10/files
And it belive it's about time to merge it.

@omsrivastava omsrivastava merged commit e2ae928 into omsrivastava:master Jan 13, 2024
@sandeepvasani
Copy link

These changes doesn't match with moment.js timezone list and breaks our integration.
E.g
Europe/Zaporozhye to Europe/Zaporizhzhia

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants