Skip to content

[18.0][MIG] pos_config_logo: Migration to version 18.0#1502

Merged
OCA-git-bot merged 4 commits intoOCA:18.0from
Tecnativa:18.0-mig-pos_config_logo
Mar 19, 2026
Merged

[18.0][MIG] pos_config_logo: Migration to version 18.0#1502
OCA-git-bot merged 4 commits intoOCA:18.0from
Tecnativa:18.0-mig-pos_config_logo

Conversation

@cristina-hidalgo-tecnativa
Copy link
Copy Markdown
Contributor

  • point_of_sale.assets bundle renamed to point_of_sale._assets_pos
  • pos_interface_section is now a <block> instead of a <div>; attrs replaced by invisible
  • _load_pos_data_fields removed — pos.config loads all fields by default, so logo is included automatically
  • Registries system replaced by patch(). PosGlobalState._loadPictures removed — logo is now injected via PosStore.getReceiptHeaderData instead
  • receipt_header.xml (new): required because Odoo 18 hardcodes the company logo URL in the ReceiptHeader template instead of using a variable, making it impossible to override from JS alone

@Tecnativa TT59136
@christian-ramos-tecnativa @pedrobaeza

Copy link
Copy Markdown
Contributor

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

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 migration pos_config_logo

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Mar 19, 2026
@OCA-git-bot OCA-git-bot mentioned this pull request Mar 19, 2026
33 tasks
Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Tested on runboat.

/ocabot merge nobump

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-1502-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 66e9098 into OCA:18.0 Mar 19, 2026
5 of 7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

@pedrobaeza pedrobaeza deleted the 18.0-mig-pos_config_logo branch March 19, 2026 09:43
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.

6 participants