Skip to content

added links to the supported calendar types#39471

Merged
Josh-Cena merged 10 commits intomdn:mainfrom
dletorey:39303-ship-temporal
May 14, 2025
Merged

added links to the supported calendar types#39471
Josh-Cena merged 10 commits intomdn:mainfrom
dletorey:39303-ship-temporal

Conversation

@dletorey
Copy link
Contributor

@dletorey dletorey commented May 9, 2025

Description

  • Added list of supported calendar types to:
    • Temporal.PlainDate.prototype.withCalendar()
    • Temporal.PlainDateTime.prototype.withCalendar()

Motivation

  • Working on the release of Temporal API and there was no rela colelation with the withCalendar() method and what calendar type can be used.

Related issues and pull requests

@dletorey dletorey requested a review from a team as a code owner May 9, 2025 14:33
@dletorey dletorey requested review from Josh-Cena and removed request for a team May 9, 2025 14:33
@github-actions github-actions bot added the Content:JS JavaScript docs label May 9, 2025
@dletorey dletorey requested a review from bsmth May 9, 2025 14:34
@github-actions github-actions bot added the size/xs [PR only] 0-5 LoC changed label May 9, 2025
@github-actions
Copy link
Contributor

github-actions bot commented May 9, 2025

Preview URLs (21 pages)

(comment last updated: 2025-05-14 16:34:40)

Copy link
Member

@Josh-Cena Josh-Cena left a comment

Choose a reason for hiding this comment

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

If you want to do this, could you make sure this link is added to the withCalendar, from, and constructor pages of every object?

My intention is for the calendarId page to be the single source of truth documenting which calendar IDs are valid for each object type, although incidentally they all match Intl.supportedValuesOf, but I don't think that's normatively required.

@github-actions github-actions bot added size/s [PR only] 6-50 LoC changed and removed size/xs [PR only] 0-5 LoC changed labels May 12, 2025
This was referenced May 14, 2025
Copy link
Member

@Josh-Cena Josh-Cena left a comment

Choose a reason for hiding this comment

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

Nice, thank you

@Josh-Cena Josh-Cena merged commit 1b77d85 into mdn:main May 14, 2025
8 checks passed
@dletorey dletorey deleted the 39303-ship-temporal branch May 19, 2025 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:JS JavaScript docs size/s [PR only] 6-50 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants