Backfill Play Store metadata for the 15 new locales - #15983
Backfill Play Store metadata for the 15 new locales#15983JorgeMucientes wants to merge 3 commits into
Conversation
Generated by 🚫 Danger |
bd6f99d to
f55797b
Compare
ee9e7fc to
6b88e16
Compare
f55797b to
a3c6534
Compare
c705a23 to
38cc38f
Compare
0299269 to
fc95647
Compare
a875711 to
7ff86ea
Compare
d7d4148 to
92dcfaf
Compare
7ff86ea to
61596db
Compare
92dcfaf to
0d7fa6e
Compare
61596db to
04db0a8
Compare
toupper
left a comment
There was a problem hiding this comment.
Thanks for the backfill, Jorge. Diff scope is clean — only the 15 new locale folders and translation-manifest.json were touched, and the existing 16 metadata directories are unmodified. Locale folder names map correctly to Google Play's expected BCP-47 codes (bg, cs-CZ, da-DK, el-GR, fi-FI, hi-IN, hu-HU, ms, no-NO, pl-PL, pt-PT, ro, th, uk, vi), and the no-NO rationale in the description matches Play's listing locale.
I ran the char-cap and prohibited-content checks across all 45 files; everything is within limits and free of HTML/Markdown leakage, prices, or "#1"/superlative claims. The manifest tracks each new entry with origin: "ai", model: claude-opus-4-7 for titles/shorts and claude-haiku-4-5 for full descriptions, plus a per-entry cache key — clear source attribution.
A handful of translation-quality nits below. These are not Play-policy blockers (the listing will publish without rejection), but most are worth fixing before the strings land in front of merchants. Flagging as suggestions and deferring to the i18n team / native speakers for the final call.
Suggestions
fastlane/metadata/android/ro/full_description.txt:15— "Transformă orice tablă" should be "Transformă orice tabletă".tablămeans blackboard, not tablet. Worth fixing before merge; this one reads as a clear mistranslation rather than a stylistic choice.fastlane/metadata/android/ro/full_description.txt:24— "datele magazinului de astazi" → "astăzi" (missing diacritic). Same line in other Romanian-script sections uses diacritics correctly, so this is a single typo.fastlane/metadata/android/fi-FI/full_description.txt:23-24— "WooCommerce rannallasi" / "suoraan rannaltasi" useranta(shore/beach), notranne(wrist). Should be "ranteessasi" / "ranteestasi". The English source says "right from your wrist", so this is a translation error rather than localization choice.fastlane/metadata/android/hi-IN/full_description.txt:15andfastlane/metadata/android/no-NO/full_description.txt:15andfastlane/metadata/android/pl-PL/full_description.txt:15— these expand "WooCommerce POS" to "WooCommerce Point of Sale". "WooCommerce POS" is the product name in English; other locales (and the rest of these same files) keep it asPOS. Worth normalizing for product-name consistency in the Play listing.fastlane/metadata/android/cs-CZ/full_description.txt:18andfastlane/metadata/android/pl-PL/full_description.txt:18—„cha-ching"opens with the Czech/Polish low-9 quote (U+201E) but closes with a straight ASCII quote (U+0022). Should be paired — either both straight"…", or properly close with U+201C („cha-ching"). Same mismatch in both locales suggests a model artifact.fastlane/metadata/android/pt-PT/full_description.txt:1— "Gerencie a sua loja" uses a Brazilian-Portuguese conjugation, while the rest of the file (lines 3, 5, 8, etc.) is consistent European Portuguese (Gira,Crie,Adicione). Consider replacing line 1's verb (e.g. "Gira a sua loja" or "Faça a gestão da sua loja") to match.fastlane/metadata/android/da-DK/short_description.txtis exactly 80 chars — right at the Play cap. Not a blocker (you're under the limit), but leaves zero headroom if the cap ever tightens or if a future copy edit nudges it. Worth flagging for the next sweep, not this PR.- All "cha-ching" instances use ASCII
"..."while the en-US source uses curly"…". Not a regression vs other existing locales (e.g.it-ITalso uses straight quotes), but if a future pass normalizes typographic quotes per locale convention, that would be the time to align these.
Positives
- Scope discipline is great: 46 files, exactly the 15 new locales + manifest, no incidental changes to existing locales or changelogs.
- Char-cap test (
woo_ai_translation_test.rb) plus the per-file local cap check are a good safety net for the title/short-description limits; I re-verified all 45 files independently and every one is comfortably within Play's caps (titles 23–28 chars, shorts 47–80, full 2141–2889). - Manifest entries are well structured with
origin,cache_key,model,prompt_version, andupdatedtimestamps — easy to trace any cell back to its generation later. - Folder naming matches Play Console's locale codes (especially the
no-NOovernbcall, which is the right choice for the listing). - No HTML/Markdown leakage, no prices/superlatives/testimonials — Play policy looks clean.
Reviewed on behalf of @toupper — drafted by Claude Code, please confirm before merging.
0d7fa6e to
14ac08c
Compare
04db0a8 to
12c2178
Compare
|
Thanks @toupper. I pushed the targeted Play metadata copy fixes you flagged. Changed:
Not changed:
|
Generates fastlane/metadata/android/<gp_locale>/{title,short_description,
full_description}.txt for the 15 new locales (pl-PL, pt-PT, cs-CZ, da-DK,
nb-NO, fi-FI, el-GR, hu-HU, ro, uk, bg, th, vi, hi-IN, ms), translated from
the en-US source via the AI engine (Opus for title/short_description, Haiku
for full_description). The existing 16 locales' human metadata is untouched
(scoped run via --gp-locales). Screenshot/promo text stays English by design.
All 15 pass the Play Store caps (title <=50, short_description <=80 code
points); the char-cap re-prompt-to-fit rescued ro/uk/bg/hi-IN/ms short
descriptions. Manifest records each field/locale (origin=ai) so future runs
reuse them.
Release notes (changelogs) are intentionally NOT translated here -- they're
version-scoped and belong to the code-freeze sweep.
12c2178 to
e63b2f4
Compare
|
Version |
|
Version |
|
Version |
|
Version |
|
Version |
Summary
Backfills Google Play Store listing metadata for the 15 new locales in the translation stack.
This PR adds
fastlane/metadata/android/<gp_locale>/{title,short_description,full_description}.txtfor:bgcs-CZda-DKel-GRfi-FIhi-INhu-HUmsno-NOpl-PLpt-PTrothukviIt also records the metadata translations in
fastlane/ai_translation/translation-manifest.json.Scope
title <= 50,short_description <= 80).no-NOis used for the Norwegian Play Store metadata directory to match the Google Play listing locale code.Review Fixes Included
astăzi.WooCommerce POSas the product term in Hindi, Norwegian, and Polish instead of expanding it toWooCommerce Point of Sale.cha-chingquotes in Czech and Polish.GerencietoGira.Stack
Base: #15974. #15982's metadata char-cap test has already been merged/folded into the base stack history.
Open stack above this PR:
Full stack map: #15961.
Test Plan
title.txtandshort_description.txt.git diff --check.translation-manifest.jsonplus new metadata files for the 15 locales.