[17.0][MIG] account_invoice_report_grouped_by_picking: Migration to 17.0 #296
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- When a return was in the original pickings it wasn't reflected in the corresponding invoices correctly. - Now we compute the total quantities of the moves recursively.
- When B2C prices was set, the orders where not properly grouped.
…iding edge cases * The grouping is fully done on the prepare method * It also computes the quantity applicable * Adapt views to B2C/B2B cleanly
… by zero when quantity is zero (OCA#101)
… in B2B when uses B2C
[FIX] account_invoice_report_grouped_by_picking: debug print remove [FIX] account_invoice_report_grouped_by_picking: remove debug variable
…also with the related SO picking Service lines that are not put on the pickings, remain on top of the invoice report due to this, but it has more sense to have them grouped in the same picking block than the rest of their sales order stockable lines. Tests and README adapted.
…change position=replace in report_invoice.xml to prevent errors
notes Avoid nasty exceptions. In the other hand it would be hard to fit the concept of sections and notes to the picking groups in which this report results. TT29507
… avoid crash test due to this Odoo commit OCA/OCB@8ca10a8
…lines for refund invoices without no returned quantities TT27550
…ngs & sales For manually created refund invoices that don't have pickings or orders, no lines were shown in the report
…ines After the patch on stock_picking_invoice_link to link the refund moves to the return pickings created before the refund. This module shows on the picking duplicated lines. With this patch the sign is correctly set and this duplication does not exist. TT39916
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-invoice-reporting-16.0/account-invoice-reporting-16.0-account_invoice_report_grouped_by_picking Translate-URL: https://translation.odoo-community.org/projects/account-invoice-reporting-16-0/account-invoice-reporting-16-0-account_invoice_report_grouped_by_picking/
…_qty missed by this PR: OCA#224 (review)
…lls to lines_grouped_by_picking
…rovement - Include context keys for avoiding mail operations overhead.
|
/ocabot migration account_invoice_report_grouped_by_picking |
|
hello @lethuthao2403 I'm merging OCA/stock-logistics-workflow#1446 you should be able to remove the unreleased dependency. |
lethuthao2403
force-pushed
the
17.0-mig-account_invoice_report_grouped_by_picking
branch
from
406f4a5 to
25f7da9
Compare
@gurneyalex the PR has been updated! Thanks. |
gurneyalex
approved these changes
Dec 9, 2023
|
/ocabot merge nobump |
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at 40918ff. Thanks a lot for contributing to OCA. ❤️ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
ref: BSRD-699