Skip to content

Comments

Update timeZoneName constant to timeZone#14468

Merged
teoli2003 merged 1 commit intomdn:mainfrom
lsolova:patch-1
Mar 30, 2022
Merged

Update timeZoneName constant to timeZone#14468
teoli2003 merged 1 commit intomdn:mainfrom
lsolova:patch-1

Conversation

@lsolova
Copy link
Contributor

@lsolova lsolova commented Mar 30, 2022

Summary

Correct timeZoneName and mention what default value of timeZone is.

Motivation

Correct misleading information. Example and description was not mentioning the same option name.

Supporting details

Related issues

Metadata

This PR…
-->

  • Adds a new document
  • Rewrites (or significantly expands) a document
  • Fixes a typo, bug, or other error

Correct `timeZoneName` and mention what default value of `timeZone` is.
@lsolova lsolova requested a review from a team as a code owner March 30, 2022 13:26
@lsolova lsolova requested review from teoli2003 and removed request for a team March 30, 2022 13:26
@github-actions github-actions bot added the Content:JS JavaScript docs label Mar 30, 2022
@github-actions
Copy link
Contributor

Copy link
Contributor

@teoli2003 teoli2003 left a comment

Choose a reason for hiding this comment

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

Thanks. Good catch!

@teoli2003 teoli2003 merged commit 07d7a6d into mdn:main Mar 30, 2022
@teoli2003
Copy link
Contributor

Thanks a lot!

Looks good to me.

Congratulations upon your first merged commit here! Welcome aboard! 🎉

@teoli2003 teoli2003 changed the title Update index.md Update timeZoneName constant to timeZone Mar 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:JS JavaScript docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants