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

Added Danish language support #343

Merged
merged 4 commits into from
Dec 16, 2016

Conversation

Andersbiha
Copy link
Contributor

Danish language support.

Since AM, PM and ordinal number notation isn't used in Danish i kept the defaults.

var weekdaysFull = ['søndag', 'mandag', 'tisdag', 'onsdag', 'torsdag', 'fredag', 'lørdag']
var meridiemUppercase = ['AM', 'PM']
var meridiemLowercase = ['am', 'pm']
var meridiemFull = ['a.m.', 'e.m.']
Copy link
Contributor

Choose a reason for hiding this comment

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

'e.m.'

@Andersbiha is it a typo?

@leshakoss
Copy link
Contributor

@Andersbiha thank you for your contribution! Please fix failing tests https://travis-ci.org/date-fns/date-fns/builds/184494140#L5775

@Andersbiha
Copy link
Contributor Author

Argh - Just realized i'd been pushing from the wrong branch.
I just repushed with the changes needed for the tests to succeed, thanks for the patience ;)

@DeMuu
Copy link

DeMuu commented Dec 16, 2016

I think the locale short name should be changed to DA

the ISO 639-2 list states the 2 letter code as DA :)

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.

Looks good to me 👍

@leshakoss
Copy link
Contributor

@Andersbiha thanks again for contribution! 🇩🇰 Danish locale was released with 1.21.0 (diff: v1.20.1...v1.21.0)

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

Successfully merging this pull request may close these issues.

3 participants