[ADD] new module l10n_fr_pos_cert_update_draft_order_line.

OCA / l10n-france

France Localization for Odoo

GNU Affero General Public License v3.0

42 stars 118 forks source link

[ADD] new module l10n_fr_pos_cert_update_draft_order_line. #482

Closed legalsylvain closed 10 months ago

legalsylvain commented 1 year ago

This module fixes the Odoo module 'French Point of Sale Certification' to allow cashier to update order lines, when order is not confirmed.

The law stipulates that the tool must not be used to modify confirmed orders, but does not specify that it is forbidden to change a quantity if the order is in the process of being seized.
If the cashier make a mistake, the current Odoo design generates hugly and incomprehensible tickets for the customer.
In fact, Odoo core module is totally inconsistent in its understanding of the law, since it prohibits the modification of a line in a draft order, but authorizes the deletion of the full draft order.

LESTRAT21 commented 11 months ago

Agree with you @legalsylvain concerning the unconsistant rule of Odoo module that enables to delete a ticket but prohibits the modification of a line in a draft order! I did a functional review and installed your new module l10n_fr_pos_cert_update_draft_order_line Thanks for this module !

legalsylvain commented 11 months ago

Hi @LESTRAT21 thanks for you review. Just a point regarding validation of PR : You can go on "File changed" tab, then click on "Review Changes", click on "Approve" and let a comment. That way, bots knows that the PR is validated. (and Repo Responsible can merge it).

legalsylvain commented 10 months ago

/ocabot merge patch

OCA-git-bot commented 10 months ago

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

OCA-git-bot commented 10 months ago

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