feat: support all IANA timezones in sync schedule - #82
Open
renoviana wants to merge 1 commit into
Open
Conversation
- Add get_all_timezones() utility listing every IANA timezone with label and current UTC offset, sorted by offset then name - Return the full timezone list from /api/timezone/supported (the dashboard expected a "timezones" field that was never returned, so the UI always fell back to a hardcoded list) - Load timezones automatically in Sync Settings and in the setup wizard, removing hardcoded lists (browser Intl fallback if the API is unavailable) - Add Brazilian abbreviations (AMT, ACT, FNT) and BR regional preferences for TZ environment variable resolution - Fix regional categorization of abbreviations: the South America bucket was unreachable and military codes leaked into continent buckets - Add unit tests and update API and configuration docs
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Adds full IANA timezone support to the sync schedule. The dashboard timezone
dropdown expected a
timezonesfield from/api/timezone/supportedthat theAPI never returned, so the UI always fell back to a small hardcoded list.
The endpoint now returns all IANA timezones (with current UTC offset), and
both the Sync Settings dropdown and the setup wizard load the list
automatically instead of using hardcoded values. Also adds Brazilian
abbreviations (AMT, ACT, FNT) with a BR regional preference for TZ
resolution, and fixes the regional categorization of abbreviations
(the South America bucket was unreachable and military codes leaked into
continent buckets).
Type of Change
Testing
Note:
npm testis referenced in CONTRIBUTING.md but the frontend has notest script configured, so no frontend tests were run.
Checklist