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

#810 Correct short and abbreviated dayValues for de locale #817

Merged
merged 1 commit into from
Jul 27, 2018

Conversation

Philipp91
Copy link
Contributor

The values are taken from https://www.unicode.org/cldr/charts/32/summary/de.html#1844, using "Days - (abbreviated|short) - Formatting".
Alternatively, the short values could be left as is (without the period). According to the CLDR table, that's the Swiss locale, but it would provide a two-letter version matching the English locale.

Copy link
Contributor

@leshakoss leshakoss left a comment

Choose a reason for hiding this comment

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

Great! Could you also please fix the test src/locale/de/test.js on line 87?

The values are taken from https://www.unicode.org/cldr/charts/32/summary/de.html#1844, using "Days - (abbreviated|short) - Formatting".
Alternatively, the short values could be left as is (without the period). According to the CLDR table, that's the Swiss locale, but it would provide a two-letter version matching the English locale.
@Philipp91
Copy link
Contributor Author

Done

Copy link
Contributor

@leshakoss leshakoss left a comment

Choose a reason for hiding this comment

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

Great, thanks!

@leshakoss leshakoss merged commit fd3ca3c into date-fns:master Jul 27, 2018
@leshakoss
Copy link
Contributor

I've changed the short values to be without the period, and released this as v2.0.0-alpha.14. Thank you for your contribution! ⭐️

@Philipp91
Copy link
Contributor Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants