[16.0][ADD] pos_screen_element_custom_size

[geomer198]

This module allows to define custom size values for certain POS screen elements.

[geomer198]

Interesting module. thanks for sharing ! could you remove the pandoc... .deb file ?

thanks !

Could you check again?

[cvinh]

Interesting module. thanks for sharing ! could you remove the pandoc... .deb file ? thanks !

Could you check again?

Do you have any idea how the pandoc deb appeared ? I had the same in one of my PR once...

[geomer198]

Interesting module. thanks for sharing ! could you remove the pandoc... .deb file ? thanks !

Could you check again?

Do you have any idea how the pandoc deb appeared ? I had the same in one of my PR once...

Pandoc is added after running pre-commit.

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[dreispt]

/ocabot merge nobump

[legalsylvain]

/ocabot merge nobump

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-1158-by-legalsylvain-bump-nobump, awaiting test results.

[OCA-git-bot]

@legalsylvain your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-1158-by-legalsylvain-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

[legalsylvain]

Hi @geomer198 could you rebase and take a look on red ci ? Thanks !

[ivs-cetmix]

hi @legalsylvain could you please check again? Thank you in advance!

[legalsylvain]

/ocabot merge nobump

[OCA-git-bot]

This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-1158-by-legalsylvain-bump-nobump, awaiting test results.

[OCA-git-bot]

@legalsylvain your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-1158-by-legalsylvain-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

[legalsylvain]

HI @OCA/pos-maintainers. we have regularly a weird error on OCA/pos CI. Description :

• The PR is green (all checks are OK)
• we call ocabot
• the merge of ocabot fails.
• when we check the errors, there is an error on OCB, meanwhile odoo is green.

• finally after many tries, it works. (without any code change, or any rebase).

Do you have an idea on how to solve the problem ?

thanks ! cc @ivantodorovich

[ivantodorovich]

Indeed, I've seen it too. I'll try to investigate in the coming days. It's always related to the pos_order_to_sale_order module, isn't it?

2024-05-10 14:45:44,264 397 ERROR odoo odoo.addons.pos_order_to_sale_order.tests.test_module: FAIL: TestUi.test_pos_order_to_sale_order
Traceback (most recent call last):
  File "/__w/pos/pos/pos_order_to_sale_order/tests/test_module.py", line 36, in test_pos_order_to_sale_order
    self.assertEqual(len(before_orders) + 1, len(after_orders))
AssertionError: 1 != 0

[ivantodorovich]

fix proposed here:

• https://github.com/OCA/pos/pull/1211

[ivantodorovich]

/ocabot merge nobump

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-1158-by-ivantodorovich-bump-nobump, awaiting test results.

[OCA-git-bot]

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