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

Fixed override of the long time format for Australian locale #180

Merged

Conversation

idpaterson
Copy link
Collaborator

@idpaterson idpaterson commented Aug 20, 2016

I had an Australian user noticing errors due to missing "short" time format. As it turned out, every format except "long" was missing. The previous implementation accidentally removed all other formats.


This change is Reviewable

Previous implementation accidentally removed all other formats.
@idpaterson idpaterson added the bug label Aug 20, 2016
@idpaterson idpaterson self-assigned this Aug 20, 2016
@codecov-io
Copy link

codecov-io commented Aug 20, 2016

Current coverage is 76.89% (diff: 100%)

Merging #180 into master will not change coverage

@@             master       #180   diff @@
==========================================
  Files            14         14          
  Lines          1567       1567          
  Methods           0          0          
  Messages          0          0          
  Branches        288        288          
==========================================
  Hits           1205       1205          
  Misses          268        268          
  Partials         94         94          

Powered by Codecov. Last update 63e65e2...9750a7d

@idpaterson idpaterson merged commit 073b91b into bear:master Aug 20, 2016
@idpaterson idpaterson added this to the 2.2 milestone Aug 24, 2016
@bear bear mentioned this pull request Aug 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants