Skip to content

Add finnish translations#49

Open
ristinolla wants to merge 3 commits into
wearebraid:masterfrom
ristinolla:add-finnish-localization
Open

Add finnish translations#49
ristinolla wants to merge 3 commits into
wearebraid:masterfrom
ristinolla:add-finnish-localization

Conversation

@ristinolla
Copy link
Copy Markdown

This adds Finnish to localizations.

Comment thread src/locales/fi.js Outdated
if ((!isNaN(value) && force !== 'length') || force === 'value') {
return `${s(name)} kenttä tulisi ${args[0]} and ${args[1]}.`
}
return `${s(name)} must be between ${args[0]} and ${args[1]} characters long.`
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Translation missing

Comment thread src/locales/fi.js Outdated
* The confirmation field does not match
*/
confirm: function ({ name, args }) {
return `${s(name)} does not match.`
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Same here, Finnish translation is missing :)

Comment thread src/locales/fi.js Outdated
* Valid accepted value.
*/
accepted: function ({ name }) {
return `Please accept the ${name}.`
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Finnish translation is missing

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