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

Migrate from omsrivastava/timezones-list to vvo/tzdb #4479

Closed
chakflying opened this issue Feb 11, 2024 · 2 comments Β· Fixed by #4521
Closed

Migrate from omsrivastava/timezones-list to vvo/tzdb #4479

chakflying opened this issue Feb 11, 2024 · 2 comments Β· Fixed by #4521
Labels
area:settings Related to Settings page and application configration feature-request Request for new features to be added help wanted May need your help to test or answer

Comments

@chakflying
Copy link
Collaborator

chakflying commented Feb 11, 2024

πŸ“‘ I have found these related issues/pull requests

none

🏷️ Feature Request Type

Other

πŸ”– Feature description

omsrivastava/timezones-list, the library currently used by our front-end, is rarely updated and has inaccuracies.

vvo/tzdb looks much better as it is automatically kept up-to-date with authoritative sources.

Some migration work is needed since the data format is different.

βœ”οΈ Solution

Migrate from omsrivastava/timezones-list to vvo/tzdb

❓ Alternatives

No response

πŸ“ Additional Context

No response

@chakflying chakflying added feature-request Request for new features to be added area:settings Related to Settings page and application configration labels Feb 11, 2024
@CommanderStorm CommanderStorm added the help wanted May need your help to test or answer label Feb 12, 2024
@louislam
Copy link
Owner

It can be considered as a bug fix, because the omsrivastava/timezones-list#14 issue you mentioned should be related to the DST issue, which timezones-list does not handling it at all.

https://www.zeitverschiebung.net/en/timezone/america--santiago

@huzaifaazim0
Copy link
Contributor

created a pull requestwith migrated lib
#4521

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:settings Related to Settings page and application configration feature-request Request for new features to be added help wanted May need your help to test or answer
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants