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

Canonicalization for Calendar identifiers? #828

Open
anba opened this issue Jun 23, 2023 · 2 comments
Open

Canonicalization for Calendar identifiers? #828

anba opened this issue Jun 23, 2023 · 2 comments
Labels
c: datetime Component: dates, times, timezones calendar s: blocked Status: the issue is blocked on upstream Temporal Issues relating to Intl support in Temporal

Comments

@anba
Copy link
Contributor

anba commented Jun 23, 2023

Previously reported at tc39/proposal-temporal#2314, but that issue was closed after the case-normalisation part was fixed.

Q: Should new Temporal.Calendar("islamicc").id return "islamicc" or the canonical identifier "islamic-civil"?

If no canonicalisation should happen, it'd be good to mention this in https://tc39.es/proposal-temporal/#sec-calendar-types.

@justingrant justingrant transferred this issue from tc39/proposal-temporal Aug 17, 2023
@justingrant
Copy link
Contributor

justingrant commented Aug 17, 2023

Temporal meeting 2023-08-17:

  • Per @sffc, we'll transfer this issue to the 402 repo. TG2 should decide whether a normative PR is needed.
  • If we want to match time zone behavior, then DTF.resolvedOptions() should retain current behavior, but testing for equality (e.g. among Temporal.PlainDate instances with "islamicc" or the canonical identifier "islamic-civil") should canonicalize first before comparing for equality.
  • There's also the cases of "gregorian" and "ethiopic-amete-alem" where those aliases are accepted in CLDR but they are resolved to the canonical IDs "gregory" and "ethioaa", respectively. Should behavior be consistent for these vs. "islamicc" ?
  • UTS 35 explains the difference between deprecated and preferred (like "islamicc" → "islamic-civil") and alias (like "gregorian" and "ethiopic-amete-alem").

@sffc sffc added c: datetime Component: dates, times, timezones s: blocked Status: the issue is blocked on upstream Temporal Issues relating to Intl support in Temporal calendar labels Aug 17, 2023
@ljharb ljharb changed the title Canonicalisation for Calendar identifiers? Canonicalization for Calendar identifiers? Aug 17, 2023
@anba
Copy link
Contributor Author

anba commented Aug 25, 2023

@sffc sffc added this to Other Issues in ECMA-402 Meeting Topics Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: datetime Component: dates, times, timezones calendar s: blocked Status: the issue is blocked on upstream Temporal Issues relating to Intl support in Temporal
Projects
Development

No branches or pull requests

3 participants