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

Create separate receipt when merge PO with different location #29561

Closed
wants to merge 2 commits into from

Conversation

dpa-odoo
Copy link
Contributor

Description of the issue/feature this PR addresses:

  • When two RFQ merge with same vendor and validate PO it create only one receipt with two different moves and destination location on receipt not correctly set so its not correct behaviour

Current behavior before PR:

  • Currently when validate PO with two RFQ merge of same vendor with different destination location so currently it will create only one receipt with two different moves.

Desired behavior after PR is merged:

  • Now when validating PO with two RFQ of same vendor with different destination location so now it will create two separate receipt and set correct destination location on receipt

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

Task : https://www.odoo.com/web#id=1915336&action=333&active_id=131&model=project.task&view_type=form&menu_id=4720

Pad : https://pad.odoo.com/p/r.aaf5d0f8def78a86805ed6da30a86e43

@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Dec 17, 2018
@C3POdoo C3POdoo added the RD research & development, internal work label Dec 17, 2018
@kirti-c kirti-c force-pushed the master-purchase-merge-po-dpa branch from 5c2a35e to 3b7dc99 Compare February 6, 2019 10:03
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses and removed CI 🤖 Robodoo has seen passing statuses labels Feb 6, 2019
@robodoo robodoo removed the CI 🤖 Robodoo has seen passing statuses label Mar 15, 2019
@parthchoksi-github parthchoksi-github force-pushed the master-purchase-merge-po-dpa branch 3 times, most recently from 82d581a to 9155f29 Compare March 27, 2019 09:41
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Mar 27, 2019
@jir-odoo jir-odoo force-pushed the master-purchase-merge-po-dpa branch from 9155f29 to b4e8774 Compare March 29, 2019 11:01
@robodoo robodoo removed the CI 🤖 Robodoo has seen passing statuses label Mar 29, 2019
@jir-odoo jir-odoo force-pushed the master-purchase-merge-po-dpa branch from aa0980d to 68e09af Compare March 29, 2019 11:43
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Apr 2, 2019
@jir-odoo jir-odoo force-pushed the master-purchase-merge-po-dpa branch from 68e09af to 333b886 Compare April 5, 2019 13:20
@robodoo robodoo removed the CI 🤖 Robodoo has seen passing statuses label Apr 5, 2019
…location is different

 - add new field location_dest_id on purchase order line
 - when merge RFQ with same vendor Ex: Product A  : receipt in 1 steps (from supplier to stock) and Product B : receipt in 2 steps (from input to stock) and confirm PO it will create two different picking instead of one Ex: Product A going to Stock and Product B going to Input

Task ID : #1915336
… different location on purchase order line

Task ID : #1915336
@jir-odoo jir-odoo force-pushed the master-purchase-merge-po-dpa branch from 333b886 to 4d073f2 Compare April 5, 2019 13:30
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Apr 5, 2019
@rdeodoo
Copy link
Contributor

rdeodoo commented Aug 6, 2019

Task cancelled

@rdeodoo rdeodoo closed this Aug 6, 2019
@robodoo robodoo added closed 💔 and removed CI 🤖 Robodoo has seen passing statuses labels Aug 6, 2019
@xmo-odoo xmo-odoo deleted the master-purchase-merge-po-dpa branch November 20, 2019 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants