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 Finnish locale #351

Merged
merged 3 commits into from
Sep 26, 2018
Merged

Add Finnish locale #351

merged 3 commits into from
Sep 26, 2018

Conversation

Haprog
Copy link
Contributor

@Haprog Haprog commented Sep 25, 2018

This adds basic support for Finnish locale but atm. proper Finnish support can't be implemented until issue #302 is fixed.

relativeTime properties are now configured for past tense usage only because I think it's a more common use case with this kind of library.

For now using future tense with this locale will result in incorrect forms and text that doesn't make sense.

This adds basic support for Finnish locale but atm. proper Finnish support can't be implemented until issue iamkun#302 is fixed.
relativeTime properties are now configured for past tense usage only because I think it's a more common use case with this kind of library.
For now using future tense with this locale will result in incorrect forms and text that doesn't make sense.
@codecov-io
Copy link

codecov-io commented Sep 25, 2018

Codecov Report

Merging #351 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #351   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          38     39    +1     
  Lines         410    413    +3     
  Branches       55     55           
=====================================
+ Hits          410    413    +3
Impacted Files Coverage Δ
src/locale/fi.js 100% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5d5725b...a61d243. Read the comment docs.

Haprog and others added 2 commits September 25, 2018 13:49
Week names and month names are not capitalized in Finnish so having them all lowercase in the locale file is intentional. They should only start with a capital in case the word is a first word in a sentence or title.
@iamkun iamkun merged commit 97712c4 into iamkun:master Sep 26, 2018
@iamkun
Copy link
Owner

iamkun commented Sep 26, 2018

@Haprog THX

@Haprog Haprog deleted the finnish-locale branch September 26, 2018 11:43
@iamkun
Copy link
Owner

iamkun commented Dec 13, 2018

🎉 This PR is included in version 1.7.8 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ttsirkia
Copy link
Contributor

@Haprog it seems that the problem with future times can be now fixed: #767

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.

4 participants