Skip to content

[18.0][FIX] document_url: Easier template inheriting#581

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
decgroupe:18.0-fix-document_url
Jan 8, 2026
Merged

[18.0][FIX] document_url: Easier template inheriting#581
OCA-git-bot merged 1 commit into
OCA:18.0from
decgroupe:18.0-fix-document_url

Conversation

@ypapouin

@ypapouin ypapouin commented Jan 8, 2026

Copy link
Copy Markdown
Contributor
  • In a first step, instead of replacing a node content with its verbatim copy, we uses the xpath "replace" command with "$0" magic keyword to move all the <t> content into a <div>.
  • In a second step, the "Add URL" button is added after the existing "Attach files"button.

- In a first step, instead of replacing a node content with its verbatim copy, we uses the xpath "replace" command with "$0" magic keyword to move all the <t> content into a <div>.
- In a second step, the "Add URL" button is added after the existing "Attach files"button.
@ypapouin ypapouin changed the title [18.0[FIX] document_url: Easier template inheriting [18.0][FIX] document_url: Easier template inheriting Jan 8, 2026
@pedrobaeza pedrobaeza added this to the 18.0 milestone Jan 8, 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.

Tested and working. Thanks for the improvement.

@marcelsavegnago marcelsavegnago 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.

LGTM

@pedrobaeza

Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 18.0-ocabot-merge-pr-581-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit a87b755 into OCA:18.0 Jan 8, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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