[16.0][IMP] pos_default_partner: Set default partner on pos generated moves

[sergiobstoj]

Assigns the default partner to the Entries

[sergiobstoj]

@OCA/pos-maintainers can this be reviewed?

[sergiobstoj]

Hi, @ivantodorovich First of all, thanks for your review.

Having read your comments, ¿it could be an idea to make it configurable what I am proposing in here? ¿Or even including it in a new addon? Overall I believe it's a good improvement.

About the tests, I added the fix for loading a CoA. Greetings.

[ivantodorovich]

Having read your comments, ¿it could be an idea to make it configurable what I am proposing in here? ¿Or even including it in a new addon? Overall I believe it's a good improvement.

I think both options are good ideas. Personally I wouldn't oppose to it being configurable. Module authors' have the last word about such topics, so we'll give them some time to catch up and review.

About the tests, I added the fix for loading a CoA.

Hey, thanks for giving it a try! It seems they're still not being executed, though.

We have an issue with our CI, that gives a false positive ✅ in such cases (tracked here: https://github.com/OCA/oca-ci/issues/105)

🧠 How to tell if they're being executed, then?

1️⃣ Well, the first and most obvious one is the coverage report:

• It's red
• Access the codecov report and shows no tests were executed: https://app.codecov.io/gh/OCA/pos/pull/1416?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=checks&utm_campaign=pr+comments&utm_term=OCA

2️⃣ Then, there's also this log line in the build logs:

2025-08-22 05:43:43,275 470 INFO odoo odoo.tests.suite: skipped setUpClass (odoo.addons.pos_default_partner.tests.test_pos_default_partner.TestPosDefaultPartner) : Accounting Tests skipped because the user's company has no chart of accounts. 

[bealdav]

Hi @ivantodorovich thanks to ping me.

FYI information I didn't use this module. I prefer to be away from POS projects. Particularly point_of_sale module which duplicate code, and workflows to make simple things complex and confusing for users i.e. pos.order vs sale.order and then all implications with update/cancellation. In 95% of the use of pos in real life, you don't need to be disconnected, then why this complexity ?

Then I'll let main contributors of this module replace me as author.

I'm sure your options will be more consistent than mine.

Thanks a lot.