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

Updated Finnish translations #2287

Merged
merged 3 commits into from
Mar 14, 2017
Merged

Updated Finnish translations #2287

merged 3 commits into from
Mar 14, 2017

Conversation

bajiat
Copy link
Contributor

@bajiat bajiat commented Mar 14, 2017

Closes #2175

@@ -556,7 +240,7 @@
"dashboardDataTable_timeStamp": "P\u00e4iv\u00e4ys",
"dashboardDataTable_title": "Analytiikka",
"dashboardDataTable_pageCount": "Sivu",
"dashboardCharts_placeholder_noDataFound": "Tietoja ei l\u00f6ytynyt. Kokeile muuttaa suodatusvaihtoehtoja n\u00e4hd\u00e4ksesi analytiikkadataa.",
"dashboardCharts_placeholder_noDataFound": "Tietoja ei l\u00f6ytynyt. <br\/> Kokeile muuttaa suodatusvaihtoehtoja n\u00e4hd\u00e4ksesi analytiikkadataa.",
Copy link
Contributor

Choose a reason for hiding this comment

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

@bajiat Delete <br\/> tag

"organizationProfile_bigText_apisCount": "Rajapinnat",
"organizationProfileHeader_editButton_text": "Muokkaa",
"organizationProfileHeader_tabs_apis": "Rajapinnat",
"organizationProfileHeader_tabs_settings": "Asetukset",
"organizationProfile_smallText_apisCount": "(Rajapintoja: %s)",
"organizationProfile_smallText_apisCount": "",
"organizationProfile_smallText_apisCount_plural": "",
Copy link
Contributor

Choose a reason for hiding this comment

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

@bajiat Why did you leave it as empty strings? It should be like for apiCatalog_apisCount and apiCatalog_apisCount_plural

@marla-singer
Copy link
Contributor

Another things look good

@bajiat
Copy link
Contributor Author

bajiat commented Mar 14, 2017

Are the lines 445 and 446 ok now? Those have somehow managed to skip my attention. We use a tool for translation and proofreading and I remember thinking that the apicount was cleverly solved, but I can't remember even seeing lines 445 and 446.

@marla-singer
Copy link
Contributor

@bajiat Yes, now it is ok. Do I merge it or do you usually merge that PRs?

@bajiat
Copy link
Contributor Author

bajiat commented Mar 14, 2017

@marla-singer Please merge this and thank you for reviewing!

@marla-singer marla-singer merged commit 2ca6e46 into develop Mar 14, 2017
@marla-singer marla-singer deleted the FI-translation-update branch March 14, 2017 12:45
@brylie brylie modified the milestone: Sprint 38 Mar 16, 2017
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

3 participants