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

Translations of privacy strings in pegasus, apps, and dashboard #22648

Merged
merged 5 commits into from May 24, 2018

Conversation

tanyaparker
Copy link
Contributor

No description provided.

@tanyaparker
Copy link
Contributor Author

@Hamms I hope this doesn't conflict with your syncs. Also, do you have a recommendation for how to add these translations to crowdin before your next sync (so they don't get overwritten?) I was going to upload all the files manually via the Crowdin UI but wondering if you have an easier way with your scripts.

Copy link
Contributor

@islemaster islemaster left a comment

Choose a reason for hiding this comment

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

Yay! Nice work Tanya!

@Hamms
Copy link
Contributor

Hamms commented May 24, 2018

Hmm, it might. Lemme merge the latest sync, and if GH doesn't have a conflict then I think we're good!

@tanyaparker
Copy link
Contributor Author

No problem. I think it'll be easy to fix anyway. I'm on it!

@Hamms
Copy link
Contributor

Hamms commented May 24, 2018

Looks like it does; how difficult will it be to re-apply these changes to staging latest?

Re; uploading translations, the crowdin CLI has an upload translations command that should work the same as uploading via UI. You can apply it like:

crowdin --config bin/i18n/codeorg_crowdin.yml --identity bin/i18n/codeorg_credentials.yml upload translations

provided you've set up credentials as outlined here

Although I'm not sure what that will do for new translations we've received since the last sync; I suspect doing an upload either with the command or manually has the chance to loose use some work unless we do it right after a sync

@tanyaparker
Copy link
Contributor Author

@Hamms Ok, I'll try that! Yes, I think uploading the professional translations will "overwrite" over volunteer translations. I think the volunteer translations would still exist, but the professional translations would take precedence because they're newer. But while I was doing this there wasn't much already translated, since the strings have only been in Crowdin a few days. I think it's ok. I'll be looking closely at your next sync to make sure we didn't lose any professional translations.

Merge conflict resolved. Ready for your thumbs up, Elijah. :)

"gdprDialogDetails": "귀하의 본 사이트 사용에 따른 데이터는 미국으로 전송되어 보관 또는 처리될 수 있습니다.",
"gdprDialogSeePrivacyPolicy": "(상세한 내용은 당사의 개인 정보 보호 정책을 참조하십시오.)",
"gdprDialogLogout": "로그 아웃",
"gdprDialogYes": "",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Here's one that was already translated. According to google translate it looks like a Yes/Yeah. I'll revert and take the existing volunteer translation.

homepage_below_hero_announcement_text: Мы соблюдаем вашу конфиденциальность и
вносим некоторые обновления в нашу политику конфиденциальности, которые вступают
в силу с 25 мая.
homepage_below_hero_announcement_link_text: Подробнее
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Second instance where the string was already translated. Professional translations is briefer (better for buttons) and according to Google Translate says "Learn More" while the volunteer translation is more like "to learn more." Will take the professional translation.

homepage_below_hero_announcement_link_text: 繼續學習
homepage_below_hero_announcement_text: 我們尊重您的隱私權,並且正在對我們的隱私權政策做一些更新
更新後的政策將於 5 月 25 日生效。
homepage_below_hero_announcement_link_text: 瞭解更多資訊
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Third instance. Volunteer translation is more like "continue studying" while professional translation is "learn more." Keeping professional translation.

@tanyaparker
Copy link
Contributor Author

tanyaparker commented May 24, 2018

There were 3 strings that already had a volunteer translation. For the Korean string, I kept the volunteer translation that said "Yes" instead of the professional translation "Yeah." The other two I kept the professional translation which seemed more accurate/terse.

@Hamms
Copy link
Contributor

Hamms commented May 24, 2018

Oh, to clarify I'm actually worried about all the other strings. I believe the upload is an all-or-nothing thing, so any strings anywhere else in our system that have been updated will be overwritten with whatever value is currently in the i18n files.

@tanyaparker tanyaparker merged commit 2048e46 into staging May 24, 2018
@tanyaparker tanyaparker deleted the privacy-strings branch May 24, 2018 19:55
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