Skip to content

[FIX] l10n_ar_account_reports: post migration script update#982

Closed
pablohmontenegro wants to merge 1 commit intoingadhoc:19.0from
adhoc-dev:19.0-h-115102-pam
Closed

[FIX] l10n_ar_account_reports: post migration script update#982
pablohmontenegro wants to merge 1 commit intoingadhoc:19.0from
adhoc-dev:19.0-h-115102-pam

Conversation

@pablohmontenegro
Copy link
Copy Markdown
Contributor

Ticket: 115102
No hace falta volver a establecer account_tags con el método _l10n_ar_account_reports_setup_account_tags

Ticket: 115102
No hace falta volver a establecer account_tags con el método _l10n_ar_account_reports_setup_account_tags
Copilot AI review requested due to automatic review settings April 13, 2026 11:55
@roboadhoc
Copy link
Copy Markdown
Contributor

Pull request status dashboard

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Este PR ajusta el script de migración post de l10n_ar_account_reports para evitar re-ejecutar la configuración de account_tags vía _l10n_ar_account_reports_setup_account_tags, alineado con el ticket 115102.

Changes:

  • Se elimina del post-migration.py la búsqueda de compañías AR y la llamada a _l10n_ar_account_reports_setup_account_tags(...) tras la limpieza.
  • La migración queda enfocada únicamente en borrar el/los registros obsoletos por xml_id y registrar el fin del proceso.

@pablohmontenegro pablohmontenegro deleted the 19.0-h-115102-pam branch April 22, 2026 19:37
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.

3 participants