Skip to content

[19.0][MIG] l10n_din5008_move_name: Migration to 19.0#212

Merged
OCA-git-bot merged 4 commits intoOCA:19.0from
mt-software-de:19.0-mig-l10n_din5008_move_name_new
Mar 19, 2026
Merged

[19.0][MIG] l10n_din5008_move_name: Migration to 19.0#212
OCA-git-bot merged 4 commits intoOCA:19.0from
mt-software-de:19.0-mig-l10n_din5008_move_name_new

Conversation

@mt-software-de
Copy link
Copy Markdown
Contributor

Since the field l10n_din5008_document_title of account.move was removed and moved into the template.
I removed the models and added the template.

grafik

@mt-software-de mt-software-de marked this pull request as draft January 9, 2026 15:19
@mt-software-de mt-software-de force-pushed the 19.0-mig-l10n_din5008_move_name_new branch from 4ecf591 to f418f79 Compare January 9, 2026 15:26
@mt-software-de
Copy link
Copy Markdown
Contributor Author

@etobella @tv-openbig maybe you can review this PR.

@mt-software-de mt-software-de marked this pull request as ready for review January 12, 2026 09:49
@mt-software-de mt-software-de force-pushed the 19.0-mig-l10n_din5008_move_name_new branch from f418f79 to 343b667 Compare January 12, 2026 17:58
@etobella
Copy link
Copy Markdown
Member

The problem is that with the current change, it is no longer this way... it has probably a span inside 🤔

@mt-software-de mt-software-de force-pushed the 19.0-mig-l10n_din5008_move_name_new branch from 343b667 to 5ebb27a Compare January 13, 2026 00:03
@mt-software-de
Copy link
Copy Markdown
Contributor Author

The problem is that with the current change, it is no longer this way... it has probably a span inside 🤔

Yes. :-)
Done

@mt-software-de
Copy link
Copy Markdown
Contributor Author

@etobella i would need the addon also now for v18. Whats your recommendation?
Should i create a new unrelated PR. Or should i create PR for v18 and then use this as base for v19, when v18 is merged?

inherit_id="l10n_din5008.report_invoice_document"
priority="100"
>
<xpath expr="//t[@t-set='din5008_document_title']/span" position="after">
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.

If you prefer direct reference.

Suggested change
<xpath expr="//t[@t-set='din5008_document_title']/span" position="after">
<xpath expr='//span[@t-if="o and o._name == &apos;account.move&apos;"]' position="after">

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

IMO. din5008_document_title should/must be in the xpath.
Its much more readable and explicit.

If you insist on it. I would change it to.
<xpath expr='//t[@t-set='din5008_document_title']//span[@t-if="o and o._name == &apos;account.move&apos;"]' position="after">

@mt-software-de mt-software-de force-pushed the 19.0-mig-l10n_din5008_move_name_new branch 2 times, most recently from 0b321b0 to f3c87fd Compare March 17, 2026 16:17
@mt-software-de mt-software-de force-pushed the 19.0-mig-l10n_din5008_move_name_new branch from f3c87fd to 96f383e Compare March 18, 2026 10:54
Copy link
Copy Markdown
Contributor

@luisDIXMIT luisDIXMIT left a comment

Choose a reason for hiding this comment

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

Code review, LGTM!

Copy link
Copy Markdown
Member

@etobella etobella left a comment

Choose a reason for hiding this comment

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

/ocabot merge nobump

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 19.0-ocabot-merge-pr-212-by-etobella-bump-nobump, awaiting test results.

@etobella
Copy link
Copy Markdown
Member

/ocabot migration l10n_din5008_move_name

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Mar 19, 2026
@OCA-git-bot OCA-git-bot mentioned this pull request Mar 19, 2026
3 tasks
@OCA-git-bot OCA-git-bot merged commit 2be952b into OCA:19.0 Mar 19, 2026
7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at d4b67cb. 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.

4 participants