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

T381334: Translations #5124

Merged
merged 16 commits into from
Jan 8, 2025
Merged

T381334: Translations #5124

merged 16 commits into from
Jan 8, 2025

Conversation

l-olson1214
Copy link
Collaborator

@l-olson1214 l-olson1214 commented Jan 6, 2025

Phabricator:
https://phabricator.wikimedia.org/T381334
https://phabricator.wikimedia.org/T382914

Notes

  • Updates numbers as in T382914 as well (but not all of them)
  • Added some pluralization where I could but had to remove some

Test Steps

  1. Scan through different languages and compare to expected strings found in doc or check variables and strings

Screenshots/Videos

@l-olson1214 l-olson1214 changed the title T381330 v2 ins T381334: Translations Jan 6, 2025
@l-olson1214 l-olson1214 requested review from a team and mazevedofs and removed request for a team January 6, 2025 20:56
golson-wmf added 2 commits January 7, 2025 10:31
# Conflicts:
#	Wikipedia/iOS Native Localizations/en.lproj/Localizable.strings
Copy link
Collaborator

@mazevedofs mazevedofs left a comment

Choose a reason for hiding this comment

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

Mostly some details on the strings, but in Spanish, Italian, and German I'm seeing the englishHoursReadingSlide, the englishTopReadSlide, and the englishEditsSlide in English instead of Spanish. In French, it was all translated (I didn't check the copy to see if it was updated though)

Wikipedia/Localizations/es.lproj/Localizable.strings Outdated Show resolved Hide resolved
Wikipedia/Localizations/de.lproj/Localizable.strings Outdated Show resolved Hide resolved
Wikipedia/Localizations/it.lproj/Localizable.strings Outdated Show resolved Hide resolved
Wikipedia/Localizations/es.lproj/Localizable.strings Outdated Show resolved Hide resolved
Wikipedia/Localizations/ja.lproj/Localizable.strings Outdated Show resolved Hide resolved
Wikipedia/Localizations/nl.lproj/Localizable.strings Outdated Show resolved Hide resolved
Wikipedia/Localizations/nl.lproj/Localizable.strings Outdated Show resolved Hide resolved
Wikipedia/Localizations/de.lproj/Localizable.strings Outdated Show resolved Hide resolved
@l-olson1214 l-olson1214 requested a review from mazevedofs January 7, 2025 20:37
Copy link
Collaborator

@mazevedofs mazevedofs left a comment

Choose a reason for hiding this comment

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

Just two occurrences of the xxx left, and since we'll be adding translations for the slides that contain En-only info in https://phabricator.wikimedia.org/T382914 we can move forward with this PR

Wikipedia/Localizations/zh-hant.lproj/Localizable.strings Outdated Show resolved Hide resolved
@l-olson1214 l-olson1214 requested a review from mazevedofs January 8, 2025 14:23
# Conflicts:
#	Wikipedia/Localizations/en.lproj/Localizable.strings
#	Wikipedia/iOS Native Localizations/en.lproj/Localizable.strings
"year-in-review-share-text" = "這是我的 Wikipedia 年度回顧,由 Wikipedia iOS 應用程式製作。 (https://apps.apple.com/app/wikipedia/id324715238) #WikipediaYearInReview";
"year-in-review-feature-explore-body" = "Wikipedia 是世界歷史上規模最大的知識資源,且免費提供給世界各地的每一個人。透過 Wikipedia 的視角,回顧 2024 年的點點滴滴。";
"year-in-review-base-saved-title" = "我們擁有超過 $1 萬個閱讀清單";
"year-in-review-base-saved-subtitle" = "今年,活躍的 iOS 應用程式用戶創建了超過 %1 萬個閱讀清單。將文章增加到閱讀清單,可以讓您即使在離線時也能存取文章。此外,您還可以登入帳戶,將閱讀清單同步到不同裝置。";
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think it should be $1 instead of %1

@mazevedofs mazevedofs merged commit f21857e into main Jan 8, 2025
4 checks passed
@mazevedofs mazevedofs deleted the T381330-v2-ins branch January 8, 2025 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants