Update functions-timezone.md #65609
Update functions-timezone.md #65609
Conversation
Improved several aspects of this text: - Removed the link given for Windows time zones because it is invalid. It is severely outdated and doesn't apply here. There is no page to link to for this. Instead, users can list the time zones on their own up-to-date system by calling `tzutil /L` - Removed the first NCRONTAB example, because it was showing a fixed UTC-5 offset without setting `WEBSITE_TIME_ZONE`. Since Eastern Time alternates between UTC-5 and UTC-4, and the whole point of this section is to show using the time zone setting, this just created confusion. - Updated other wording to be consistent with industry terminology and improve clarity.
ktoliver
added
aq-pr-triaged
|
Thanks @mj1856 for your contribution. It's good to have this reviewed/fixed by a time zone nerd (which I'm not). 😉 |
|
@ggailey777: I'm sorry - only the author of this article, @, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias. |
|
#sign-off |
|
@ggailey777: I'm sorry - only the author of this article, @, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias. |
|
@ktoliver or @GitHubber17 can you please merge this PR? I've added the missing metadata to this include to indicate that I'm the owner, but it still won't let me #sign-off. |
|
@ggailey777: I'm sorry - only the author of this article, @, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias. |
|
Hi @ggailey777 - I don't see a commit to add metadata in this PR. I'll merge this PR and you can check on it or update as needed in master (after the public to private sync). Thank you. |
|
@ktoliver i did that PR in private and it hadn't synced yet. Thanks! |
Improved several aspects of this text:
Removed the link given for Windows time zones because it is invalid. It is severely outdated and doesn't apply here. There is no page to link to for this. Instead, users can list the time zones on their own up-to-date system by calling
tzutil /LRemoved the first NCRONTAB example, because it was showing a fixed UTC-5 offset without setting
WEBSITE_TIME_ZONE. Since Eastern Time alternates between UTC-5 and UTC-4, and the whole point of this section is to show using the time zone setting, this just created confusion.Updated other wording to be consistent with industry terminology and improve clarity.