Skip to content

[17.0][IMP] l10n_es_atc: New mechanism to check for JAR updates - #4857

Merged
OCA-git-bot merged 2 commits into
OCA:17.0from
Tecnativa:17.0-l10n_es_atc-mod417-update-jar-2026
Feb 24, 2026
Merged

[17.0][IMP] l10n_es_atc: New mechanism to check for JAR updates#4857
OCA-git-bot merged 2 commits into
OCA:17.0from
Tecnativa:17.0-l10n_es_atc-mod417-update-jar-2026

Conversation

@carlos-lopez-tecnativa

Copy link
Copy Markdown
Contributor

l10n_es_atc_mod417: Update JAR to v2.3.1
TT61136

@Tecnativa @pedrobaeza @sergio-teruel @christian-ramos-tecnativa @carlosdauden could you please review this?

Check the headers to verify whether a new file exists before downloading it
@christian-ramos-tecnativa

Copy link
Copy Markdown
Contributor

El archivo parece tener versionado m417[v231]e26-zip no cambiará la url igualmente cada vez que la actualicen?

@carlos-lopez-tecnativa

Copy link
Copy Markdown
Contributor Author

El archivo parece tener versionado m417[v231]e26-zip no cambiará la url igualmente cada vez que la actualicen?

Sí, claro, la URL cambiará siempre que haya una nueva versión, pero el contenido del zip, el archivo, se llama igual.

@christian-ramos-tecnativa christian-ramos-tecnativa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ya leí la conversación en el otro PR, pensaba que esto era para coger la url de forma dinámica, ahora entiendo que es puramente para actualizar el adjunto. LGTM!

@carlos-lopez-tecnativa

Copy link
Copy Markdown
Contributor Author

Ya leí la conversación en el otro PR, pensaba que esto era para coger la url de forma dinámica, ahora entiendo que es puramente para actualizar el adjunto. LGTM!

Así es, solo para ver si hay una nueva versión, ya que antes se tenía que agregar un script de migración para borrar el adjunto anterior. Pero con este nuevo mecanismo ya no será necesario.

@sergio-teruel sergio-teruel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@pedrobaeza pedrobaeza added this to the 17.0 milestone Feb 24, 2026

@pedrobaeza pedrobaeza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 17.0-ocabot-merge-pr-4857-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 6ba2d73 into OCA:17.0 Feb 24, 2026
5 of 7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at e32c4b6. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants