-
-
Notifications
You must be signed in to change notification settings - Fork 658
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
[18.0][MIG] purchase_stock_picking_invoice_link #1793
Open
JasminSForgeFlow
wants to merge
32
commits into
OCA:18.0
Choose a base branch
from
ForgeFlow:18.0-mig-purchase_stock_picking_invoice_link
base: 18.0
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[18.0][MIG] purchase_stock_picking_invoice_link #1793
JasminSForgeFlow
wants to merge
32
commits into
OCA:18.0
from
ForgeFlow:18.0-mig-purchase_stock_picking_invoice_link
+1,082
−0
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
…in stock_picking_invoice_link
Steps to reproduce the problem: - Create a sales order with Dropshipping route. - Confirm the sale order. - Confirm the purchase order and validate the DS picking. - Invoice the sales order. - Invoice the purchase order. Current behavior: The vendor bill is not linked to the DS picking Expected behavior: It's linked. That's because 2 problems: - Current algorithm looks for existing invoices linked to the picking for discarding double links, but it's not having into account that such invoices can be customer ones. We simply filter for vendor bills in the algorithm to fix it. Twin for the customer part already done in OCA#1036. - The location filter must use the supplier location as reference, not the internal one, as in DS, there's no internal location. TT38628
…ckorders Before this patch, when we receive products in a backorder and the other quantities received where invoiced yet, the picking of the backorder will be related to the invoice created for the other quantities. With this patch, this situation will just happen when the invoicing policy of the product is based on quantities purchased. Steps to reproduce the problem: 1. Create a purchase order buying 10 units of some stockable product 2. Receive 8 units of the product 3. Create back order 4. Invoice the received quantity (8 units) 5. Receive 2 units of the product in the created back order 6. You will see that the back order picking is related to the invoice with 8 units
…nvoice If we make partial invoices, we need that all partial invoices were linked to the pickings related. Steps to reproduce the problem: 1. Create a purchase order buying 10 units of some stockable product 2. Receive all 3. Create new invoice, invoicing just 8 units (This invoice will be linked to the picking) 4. Create new invoice, invoicing the remaining quantity 5. The invoice with 2 units won't be linked to the picking x
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: stock-logistics-workflow-16.0/stock-logistics-workflow-16.0-purchase_stock_picking_invoice_link Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-16-0/stock-logistics-workflow-16-0-purchase_stock_picking_invoice_link/
Currently translated at 100.0% (2 of 2 strings) Translation: stock-logistics-workflow-16.0/stock-logistics-workflow-16.0-purchase_stock_picking_invoice_link Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-16-0/stock-logistics-workflow-16-0-purchase_stock_picking_invoice_link/hr/
Currently translated at 100.0% (2 of 2 strings) Translation: stock-logistics-workflow-16.0/stock-logistics-workflow-16.0-purchase_stock_picking_invoice_link Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-16-0/stock-logistics-workflow-16-0-purchase_stock_picking_invoice_link/es/
Currently translated at 100.0% (2 of 2 strings) Translation: stock-logistics-workflow-16.0/stock-logistics-workflow-16.0-purchase_stock_picking_invoice_link Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-16-0/stock-logistics-workflow-16-0-purchase_stock_picking_invoice_link/es/
Currently translated at 100.0% (2 of 2 strings) Translation: stock-logistics-workflow-16.0/stock-logistics-workflow-16.0-purchase_stock_picking_invoice_link Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-16-0/stock-logistics-workflow-16-0-purchase_stock_picking_invoice_link/it/
Currently translated at 100.0% (2 of 2 strings) Translation: stock-logistics-workflow-16.0/stock-logistics-workflow-16.0-purchase_stock_picking_invoice_link Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-16-0/stock-logistics-workflow-16-0-purchase_stock_picking_invoice_link/pt_BR/
Currently translated at 100.0% (2 of 2 strings) Translation: stock-logistics-workflow-16.0/stock-logistics-workflow-16.0-purchase_stock_picking_invoice_link Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-16-0/stock-logistics-workflow-16-0-purchase_stock_picking_invoice_link/fr/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: stock-logistics-workflow-17.0/stock-logistics-workflow-17.0-purchase_stock_picking_invoice_link Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-17-0/stock-logistics-workflow-17-0-purchase_stock_picking_invoice_link/
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.
Standard Migration
@ForgeFlow
Depends on: