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

Add Bengali locale (bn) for v2 #845

Merged
merged 12 commits into from Sep 18, 2018
Merged

Conversation

touhidrahman
Copy link
Contributor

@touhidrahman touhidrahman commented Aug 19, 2018

Fixes #836
This PR will introduce Bengali (bn) locale in date-fns.

@touhidrahman touhidrahman changed the title Add Bengali locale (bn) Add Bengali locale (bn) to date-fns v2 Aug 22, 2018
@touhidrahman touhidrahman changed the title Add Bengali locale (bn) to date-fns v2 Add Bengali locale (bn) for v2 Aug 22, 2018
@sh4hids
Copy link

sh4hids commented Sep 14, 2018

Thanks for such a nice work. If possible please fix these spells:
বেশী - বেশি
ঘন্টা - ঘণ্টা

@touhidrahman
Copy link
Contributor Author

@sh4hids Thank you for your review comment. AFAIK, bangla Academy prefers ন over ণ nowadays. And with বেশি, I agree with you as Bangla Academy is trying to move from ী to ি as much as possible. However IMO this is a minor issue, let's wait for the merge first. Or you can go ahead and change the বেশী to বেশি।

Copy link
Member

@kossnocorp kossnocorp left a comment

Choose a reason for hiding this comment

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

That is looking good to me, thank you for your work ❤️

@kossnocorp kossnocorp merged commit 1bb52f4 into date-fns:master Sep 18, 2018
@kossnocorp
Copy link
Member

I'm going to go through pending locales and then release a new v2 version. Thank you again, you guys are great!

@kossnocorp
Copy link
Member

I've just published a new version with this locale date-fns@2.0.0-alpha.18. See the change log: https://gist.github.com/kossnocorp/a307a464760b405bb78ef5020a4ab136#v200-alpha18

Thank you again for your work!

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.

None yet

4 participants