Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

[fi] Fixed #5554 for Finnish translation #5557

Merged
merged 2 commits into from
Oct 25, 2013
Merged

Conversation

jukkah
Copy link
Contributor

@jukkah jukkah commented Oct 17, 2013

Fixed translations of Node's name. It was translated as a regular word.

PS. Sorry for commit 6269015. It is not concerning the issue of this pull request. 😕

Edit. Reference link to #5554.

@ghost ghost assigned jasonsanjose and couzteau Oct 18, 2013
@couzteau couzteau mentioned this pull request Oct 22, 2013
@jasonsanjose
Copy link
Member

@valtlait are you available for a translation review?

@marcelgerber
Copy link
Contributor

Shouldn't we just use Node instead of Noden? Or isn't that possible in finnish?

@valtlai
Copy link
Contributor

valtlai commented Oct 23, 2013

@SAplayer In this context we must use genetive. N is like 's in end of English genetives.
@jukkah Eikös tuo Node-sana kuuluisi kirjoittaa pienellä alkukirjaimella? Muuten korjauksesi ovat erittäin hyviä.

@jukkah
Copy link
Contributor Author

jukkah commented Oct 23, 2013

@valtlait Nyt kyllä meni vaikeaksi. Node.js:n virallisella etusivulla on title-tagissa node.js, mutta muualla näyttäisi olevan Node.js. Oletuksena erisnimet alkavat isolla kirjaimella paitsi, jos erikseen halutaan tosin. Asiaa helpottaakseen englannin kielessä näyttäisi olevan melko yleistä kirjoittaa sanoja isolla alkukirjaimella korostustarkoituksessa, ota siitä sitten selvää oikeasta kirjoitusasusta. Eli summa summarum, ollaan laiskoja ja annetaan tämän olla vielä näin. Minä otan tehtäväkseni selvittää virallisen kirjoitusasun muutaman päivän sisällä. Vahvista vielä toisillekin, että käännös on kunnossa, ettei jää minun sanani varaan.

I will check one thing in ae6146c and inform you when I'm ready. It should take a day or two to complete.

@jukkah
Copy link
Contributor Author

jukkah commented Oct 24, 2013

@valtlait Virallinen kirjoitusasu on Node.js tai pelkkä Node. https://github.com/joyent/node/wiki/FAQ#what-is-the-official-spellingcapitalizationpronunciation

@couzteau I'm ready now. No updates needed.

@couzteau
Copy link
Member

@jasonsanjose Has @valtlait approved? Good to merge?

@couzteau
Copy link
Member

@valtlait can you confirm it's good to merge?

@valtlai
Copy link
Contributor

valtlai commented Oct 25, 2013

@couzteau Yes, I confirm.

@couzteau
Copy link
Member

Thanks you @valtlait . merging

couzteau added a commit that referenced this pull request Oct 25, 2013
[fi] Fixed #5554 for Finnish translation
@couzteau couzteau merged commit 7af0ab0 into adobe:master Oct 25, 2013
@jukkah jukkah deleted the iss#5554 branch November 1, 2013 16:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants