Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FW][FIX] point_of_sale: do not reconcile 0 sum lines #48799

Closed

Conversation

fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Apr 1, 2020

  • Have a clean V13 db with point_of_sale and account_accountant
  • General Settings => Accounting => Check "Anglo-Saxon Accounting"
    (debug mode)
  • Choose or create a storable product P with category C
  • Go to category C and set 'inventory valuation' to 'automated'
  • Open a new POS session PS
  • Sell product P with qty 1
  • Pay
  • New order
  • Sell product P with qty -1 (refund)
  • Pay
  • Close session PS
  • Validate session PS

Error "You are trying to reconcile some entries that are already
reconciled." will popup because the 0 sum AML is already marked as
reconciled. This can be fixed by avoiding asking reconciliation when the
amount_residual is 0.

opw-2177881

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Forward-Port-Of: #48696

- Have a clean V13 db with point_of_sale and account_accountant
- General Settings => Accounting => Check "Anglo-Saxon Accounting"
(debug mode)
- Choose or create a storable product P with category C
- Go to category C and set 'inventory valuation' to 'automated'
- Open a new POS session PS
- Sell product P with qty 1
- Pay
- New order
- Sell product P with qty -1 (refund)
- Pay
- Close session PS
- Validate session PS

Error "You are trying to reconcile some entries that are already
reconciled." will popup because the 0 sum AML is already marked as
reconciled. This can be fixed by avoiding asking reconciliation when the
amount_residual is 0.

opw-2177881

X-original-commit: 2c5ef4e
@robodoo robodoo added forwardport This PR was created by @fw-bot seen 🙂 labels Apr 1, 2020
@fw-bot
Copy link
Contributor Author

fw-bot commented Apr 1, 2020

Ping @agr-odoo, @nim-odoo
This PR targets master and is the last of the forward-port chain containing:

To merge the full chain, say

@fw-bot r+

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label Apr 1, 2020
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Apr 1, 2020
@nim-odoo
Copy link
Contributor

nim-odoo commented Apr 2, 2020

@fw-bot r+

@robodoo robodoo added the r+ 👌 label Apr 2, 2020
robodoo pushed a commit that referenced this pull request Apr 2, 2020
- Have a clean V13 db with point_of_sale and account_accountant
- General Settings => Accounting => Check "Anglo-Saxon Accounting"
(debug mode)
- Choose or create a storable product P with category C
- Go to category C and set 'inventory valuation' to 'automated'
- Open a new POS session PS
- Sell product P with qty 1
- Pay
- New order
- Sell product P with qty -1 (refund)
- Pay
- Close session PS
- Validate session PS

Error "You are trying to reconcile some entries that are already
reconciled." will popup because the 0 sum AML is already marked as
reconciled. This can be fixed by avoiding asking reconciliation when the
amount_residual is 0.

opw-2177881

closes #48799

X-original-commit: 2c5ef4e
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
@robodoo robodoo closed this Apr 2, 2020
@robodoo robodoo temporarily deployed to merge April 2, 2020 13:14 Inactive
@fw-bot fw-bot deleted the master-13.0-opw-2177881-fix-agr-Adsq-fw branch April 16, 2020 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses forwardport This PR was created by @fw-bot OE the report is linked to a support ticket (opw-...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants