Thanks to visit codestin.com
Credit goes to Github.com

Skip to content

Comments

[18.0][ADD] pos_sale_picking_keep: Keep sale pickings from PoS#1470

Open
pedrobaeza wants to merge 1 commit intoOCA:18.0from
Tecnativa:18.0-add-pos_sale_picking_keep
Open

[18.0][ADD] pos_sale_picking_keep: Keep sale pickings from PoS#1470
pedrobaeza wants to merge 1 commit intoOCA:18.0from
Tecnativa:18.0-add-pos_sale_picking_keep

Conversation

@pedrobaeza
Copy link
Member

This module inhibits the manipulation that the point of sale mades over the sales orders pickings, and the creation of new pickings under the PoS picking type.

Use Cases / Context

In some scenarios, you may not want that the point of sale (PoS) handles the pickings of the products you are paying:

  • Using the PoS as a pure payment terminal.
  • When complex stock flows are not supplied by the PoS.

In that cases, it's better to keep the original sales pickings.

TODO: Tests

@Tecnativa TT60048

@pedrobaeza pedrobaeza added this to the 18.0 milestone Jan 4, 2026
@pedrobaeza pedrobaeza force-pushed the 18.0-add-pos_sale_picking_keep branch 3 times, most recently from ba222bf to fc8647f Compare January 5, 2026 17:50
Copy link

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

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

Makes sense.

thanks !

This module inhibits the manipulation that the point of sale mades over
the sales orders pickings, and the creation of new pickings under the
PoS picking type.

Use Cases / Context
===================

In some scenarios, you may not want that the point of sale (PoS) handles
the pickings of the products you are paying:

- Using the PoS as a pure payment terminal.
- When complex stock flows are not supplied by the PoS.

In that cases, it's better to keep the original sales pickings.
@victoralmau victoralmau force-pushed the 18.0-add-pos_sale_picking_keep branch from fc8647f to 537477d Compare February 11, 2026 16:07
Copy link
Member

@victoralmau victoralmau 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 OK

@pedrobaeza
Copy link
Member Author

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

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

OCA-git-bot added a commit that referenced this pull request Feb 11, 2026
Signed-off-by pedrobaeza
@OCA-git-bot
Copy link
Contributor

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 18.0-ocabot-merge-pr-1470-by-pedrobaeza-bump-patch.

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.

@victoralmau
Copy link
Member

Test error will be fixed with odoo/odoo#241122

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants