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

CLDR-18074 Add specific-non-location -> specific location fallback #4183

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

robertbastian
Copy link
Member

@robertbastian robertbastian commented Nov 7, 2024

CLDR-18074

  • This PR completes the ticket.

Also CLDR-18084

ALLOW_MANY_COMMITS=true

@jira-pull-request-webhook
Copy link

Hooray! The files in the branch are the same across the force-push. 😃

~ Your Friendly Jira-GitHub PR Checker Bot

@robertbastian robertbastian removed their assignment Nov 7, 2024
1. America/Buenos_Aires → "Buenos Aires Time"
* America/Buenos_Aires → "Buenos Aires Time" // multiple zones in AR
* Asia/Shanghai → "China Time" // Asia/Shanghai is the _primaryZone_ for CN
* Asia/Shanghai → "China Summer Time" // daylight
Copy link
Member

Choose a reason for hiding this comment

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

This all looks good, with one caveat. In the examples, it looks like Asia/Shanghai can arbitrarily produce two different values. I think there is an implicit additional parameter, which we can add to make it clear what is happening. (Might also reformat as a table in that case, for clarity, and to have the comment in a separate column.)

Copy link
Member Author

Choose a reason for hiding this comment

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

done

@jira-pull-request-webhook
Copy link

Hooray! The files in the branch are the same across the force-push. 😃

~ Your Friendly Jira-GitHub PR Checker Bot

@macchiati macchiati merged commit 3853041 into unicode-org:main Jan 23, 2025
9 checks passed
@robertbastian robertbastian deleted the ldmltz branch January 23, 2025 00:31
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.

2 participants