Skip to content
This repository has been archived by the owner on Oct 1, 2024. It is now read-only.

Allow hour12 true to be respected by formatDate #1299

Merged
merged 1 commit into from
Feb 27, 2020

Conversation

carysmills
Copy link
Contributor

@carysmills carysmills commented Feb 26, 2020

Description

Note: a beta version of this change is already being used in web

Allows hour12 to be set to true.
Without this change, we were overwriting hour12 to be false, even when it was set to true. That meant using it to format hours was not working.

Before:
Screen Shot 2020-02-26 at 4 20 08 PM

After:
Screen Shot 2020-02-26 at 4 19 31 PM

Type of change

  • Dates Patch: Bug/ Documentation fix (non-breaking change which fixes an issue or adds documentation)

Checklist

  • I have added a changelog entry, prefixed by the type of change noted above

@carysmills carysmills merged commit 1ac3f03 into master Feb 27, 2020
@michenly michenly temporarily deployed to production February 27, 2020 20:05 Inactive
@helloneele helloneele deployed to test-use-memoized-number-format March 6, 2020 19:25 Active
@TayKangSheng TayKangSheng deployed to expand_locale_support_for_shopify_address March 9, 2020 08:16 Active
@michenly michenly temporarily deployed to gem March 24, 2020 21:43 Inactive
@meunierd meunierd deleted the dates-formatting-12-hr branch April 7, 2020 17:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants