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

[IMP/FIX] repair,sale_stock: repair improvements #162070

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mohammadhossam
Copy link
Contributor

When a picking is selected as the return order of a repair order, pre-fill partner_id and product_qty fields from the picking.

ENT PR: odoo/enterprise#60801

Task-3848611


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

@mohammadhossam mohammadhossam changed the title [IMP] repair: auto fill customer and quantity from return order [IMP] repair: repair improvements Apr 16, 2024
@robodoo
Copy link
Contributor

robodoo commented Apr 16, 2024

@C3POdoo C3POdoo added the RD research & development, internal work label Apr 16, 2024
@mohammadhossam mohammadhossam changed the title [IMP] repair: repair improvements [IMP] repair,sale_stock: repair improvements Apr 16, 2024
@mohammadhossam mohammadhossam changed the title [IMP] repair,sale_stock: repair improvements [IMP/FIX] repair,sale_stock: repair improvements Apr 16, 2024
@mohammadhossam mohammadhossam force-pushed the master-repair-improvements-abdu branch from 4b913db to 1e8b99f Compare April 18, 2024 13:17
When a picking is selected as the return order of a repair order,
pre-fill `partner_id, `product_qty` & `lot_id` fields from the picking.
Also, only lot/serial numbers from the original return transfer are
shown to the user to select from.

Task-3848611
Bug:
Invoices created for sales orders generated by a repair order didn't
show SN/lot number for products that are tracked when activating
"Display Lots & Serial Numbers on Invoices" setting. This was because
the method generating the lots for the report only included stock move
lines that had `customer` in its source or destination location usage,
which is not the case in the move lines of a repair order.

Fix:
Exclude repair move lines from this check.

Task-3848611
@mohammadhossam mohammadhossam force-pushed the master-repair-improvements-abdu branch from 1e8b99f to 29c6dbb Compare May 6, 2024 07:47
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

3 participants