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

Merge finalized release/19.8 #18612

Merged
merged 4 commits into from
May 16, 2022
Merged

Merge finalized release/19.8 #18612

merged 4 commits into from
May 16, 2022

Conversation

AliSoftware
Copy link
Contributor

  • Fastlane localized release_notes.txt updates — though most of them came in during the last beta merged yesterday
  • Localized .strings updates — though most of them came in during the last beta merged yesterday
  • Build number bump in version .xcconfig file
  • Diffs from the PRs that landed on the release branch since the last betaNone

You'll notice that the localized metadata for es-MX was deleted.

Indeed, when I ran `fastlane finalize_release`, this was in the logs
[17:58:57]: ---------------------------------
[17:58:57]: --- Step: gp_downloadmetadata ---
[17:58:57]: ---------------------------------
[17:58:57]: Project URL: https://translate.wordpress.org/projects/apps/ios/release-notes/
[17:58:57]: Locales: {"ar"=>"ar-SA", "da"=>"da", "de"=>"de-DE", "en-au"=>"en-AU", "en-ca"=>"en-CA", "en-gb"=>"en-GB", "es"=>"es-ES", "es-mx"=>"es-MX", "fr"=>"fr-FR", "he"=>"he", "id"=>"id", "it"=>"it", "ja"=>"ja", "ko"=>"ko", "nl"=>"nl-NL", "nb"=>"no", "pt-br"=>"pt-BR", "pt"=>"pt-PT", "ru"=>"ru", "sv"=>"sv", "th"=>"th", "tr"=>"tr", "zh-cn"=>"zh-Hans", "zh-tw"=>"zh-Hant"}
[17:58:57]: Source locale: -
[17:58:57]: Path: /Users/olivier/Documents/Dev/apps/WordPress-iOS/fastlane/metadata
Downloading language: ar-SA
Downloading language: da
Downloading language: de-DE
Downloading language: en-AU
Downloading language: en-CA
Downloading language: en-GB
Downloading language: es-ES
Downloading language: es-MX
[17:59:05]: No translation available for es-MX
Downloading language: fr-FR
Downloading language: he
Downloading language: id
Downloading language: it
Downloading language: ja
Downloading language: ko
Downloading language: nl-NL
Downloading language: no
Downloading language: pt-BR
Downloading language: pt-PT
Downloading language: ru
Downloading language: sv
Downloading language: th
Downloading language: tr
Downloading language: zh-Hans
Downloading language: zh-Hant

I'm not sure what happened here, because if I look in GlotPress, the translations for those are present, so no sure why the logs said "No translation available for es-MX" after all.

My best guess is that we encountered the "429 Too Many Requests" error from GlotPress again, except on gp_downloadmetadata this time; and likely our gp_downloadmetadata action just prints a generic "no translation available for …" message for anything other than 200 OK regardless of the actual error status code (likely 429 here)

By the time I realized that (i.e. when I submitted the App for review and when I opened this PR) it was already too late, and also I'm wayyyy past EOD already, so I figured I'd leave it that way for this cycle — which means es-MX will have the app title in English in AppStore for a sprint and it will be back to Mexican Spanish translation next sprint. 🤷

@AliSoftware AliSoftware added this to the 19.8 milestone May 13, 2022
@AliSoftware AliSoftware requested review from mokagio and a team May 13, 2022 18:44
@AliSoftware AliSoftware self-assigned this May 13, 2022
@peril-wordpress-mobile
Copy link

Messages
📖 This PR has the 'Releases' label: some checks will be skipped.

Generated by 🚫 dangerJS

@@ -1 +0,0 @@
WordPress - Constructor web
Copy link
Contributor Author

Choose a reason for hiding this comment

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

My guess is this was deleted because of another case of 429 Too Many Requests on es-MX, and gp_downloadmetadata treating any error without distinction as "no translation", hence this quite silent deletion during fastlane finalize_release.

See PR description for details.

@@ -1 +0,0 @@
Diseña un sitio, crea un blog
Copy link
Contributor Author

Choose a reason for hiding this comment

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

See PR description for guess at why that was deleted.

@@ -1 +0,0 @@
bloguero,escritura,bloguear,web,creador,online,tienda,negocio,construir,crear,escribir,blogs
Copy link
Contributor Author

Choose a reason for hiding this comment

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

See PR description for guess at why that was deleted.

Comment on lines -1 to -12
Crea y gestiona tu web o blog WordPress directamente desde tu dispositivo iOS: crea y edita cualquier entrada o página, sube tus fotos y videos favoritos, accede a las estadísticas o responde a los comentarios fácilmente.

Con WordPress para iOS, podrás publicar todo tu contenido desde la palma de tu mano. Escribe desde tu sofá el borrador de ese haiku que no te quitas de la cabeza. Sube y comparte en un momento esa increíble foto que hiciste durante el descanso en el trabajo. Responde a los últimos comentarios que te hayan dejado o descubre a qué nuevos países ha llegado hoy tu contenido echándole un vistazo a la sección de estadísticas.

WordPress para iOS es un proyecto de código abierto, lo que significa que tú también puedes participar en su desarrollo. Descubre más información en https://apps.wordpress.com/contribute/.

WordPress para iOS es compatible con sitios de WordPress.com y sitios de WordPress.org autoalojados que tengan WordPress 4.0 o superior.

¿Necesitas ayuda con la aplicación? Entra en el foro de ayuda https://ios.forums.wordpress.org/ o déjanos un tweet en @WordPressiOS.

Mira el aviso de privacidad para usuarios de California en https://automattic.com/privacy/#california-consumer-privacy-act-ccpa.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

See PR description for guess at why that was deleted.

@AliSoftware AliSoftware enabled auto-merge May 13, 2022 18:49
@wpmobilebot
Copy link
Contributor

You can test the changes in Jetpack from this Pull Request by:
  • Clicking here or scanning the QR code below to access App Center
  • Then installing the build number pr18612-75641f0 on your iPhone

If you need access to App Center, please ask a maintainer to add you.

@wpmobilebot
Copy link
Contributor

You can test the changes in WordPress from this Pull Request by:
  • Clicking here or scanning the QR code below to access App Center
  • Then installing the build number pr18612-75641f0 on your iPhone

If you need access to App Center, please ask a maintainer to add you.

Copy link
Contributor

@mokagio mokagio left a comment

Choose a reason for hiding this comment

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

Thank you for looking over this @AliSoftware 👍

+1 on the es-MX compromise. Our "tight" release cadence lets us swallow those kind of things because they won't be around for long anyway.

@AliSoftware AliSoftware merged commit 1f88dc1 into trunk May 16, 2022
@AliSoftware AliSoftware deleted the release/19.8 branch May 16, 2022 01:53
@mokagio mokagio mentioned this pull request May 17, 2022
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants