Skip to content

ci(mobile): novedades de las tiendas desde mobile/whatsnew, no desde la PR - #263

Merged
vgpastor merged 1 commit into
mainfrom
ci/whatsnew-from-files
Nov 3, 2026
Merged

vgpastor merged 1 commit into
mainfrom
ci/whatsnew-from-files

Conversation

@vgpastor

@vgpastor vgpastor commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Problema

La 1.0.12 salió a Play (producción) y a revisión de App Store con «Novedades» copiadas del título y el cuerpo de la PR #262, que eran texto interno de CI. generate-whatsnew.sh tomaba las notas de la PR fusionada o del cuerpo del GitHub Release, y en iOS además ponía el mismo texto en todos los idiomas.

Ya corregido en las tiendas, fuera de esta PR, el 3-nov:

  • App Store: se canceló el envío, se pusieron notas por idioma («Mejoras de estabilidad y rendimiento.» / «Stability and performance improvements.») y se reenvió. La versión está en WAITING_FOR_REVIEW con AFTER_APPROVAL.
  • Play: se actualizaron las notas del release 10012 en producción con un workflow temporal en una rama, que ya está borrada.

Cambios

  • Se borran generate-whatsnew.sh y sus tres pasos. Las notas salen solo de mobile/whatsnew/whatsnew-es-ES y whatsnew-en-US, que se versionan junto con el cambio de versión.
  • app-store-submit.mjs: con RELEASE_NOTES_DIR cada localización recibe su propio whatsnew-<locale>. Una localización sin fichero propio recibe el de es-ES.
  • ci.yml (Quality Checks):
    • Falla si un whatsnew-* pasa de 500 caracteres. Antes el error salía al subir a Play, tras el build.
    • Ejecuta los tres escenarios del mock de App Store, que no se ejecutaba en ningún workflow.
  • AGENTS.md: añade al flujo de release el paso de escribir las novedades.

Pruebas

  • Mock: stale, other-409 y newer-editable OK. Con el fallo anterior reintroducido (mismo texto en todos los idiomas), stale y other-409 fallan.
  • prettier --check y node --check en verde.
  • Revisión independiente (bugs y Clean Code): sin hallazgos bloqueantes. Las observaciones menores sobre el límite de 500 y el mock fuera del CI están aplicadas.

…la PR

Las notas de Play y App Store salian del titulo y cuerpo de la PR fusionada,
asi que la 1.0.12 publico texto interno de CI. Ahora salen solo de
mobile/whatsnew/whatsnew-<locale>, cada idioma con su texto, y el CI comprueba
el limite de 500 caracteres de Play y ejecuta el mock del envio a App Store.
@vercel

vercel Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
dea-map Ready Ready Preview Nov 3, 2026 6:08pm UTC

Request Review

@vgpastor
vgpastor merged commit 49c1f7a into main Nov 3, 2026
17 of 18 checks passed
@vgpastor
vgpastor deleted the ci/whatsnew-from-files branch November 3, 2026 18:16

This branch was successfully deployed

1 active deployment
Preview — 055b2130 Deployed Nov 3, 2026 by vercel[bot]
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.

1 participant