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.

Sign up for GitHub

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

Jump to bottom

[16.0][fix] rma: force to use the same lot through rma.operation #495

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from
Open

[16.0][fix] rma: force to use the same lot through rma.operation #495

wants to merge 1 commit into from

Conversation

JordiBForgeFlow
Copy link
Contributor

No description provided.

@JordiBForgeFlow JordiBForgeFlow force-pushed the 16.0-fix-rma-get_available_quantity branch from c225d4f to b53a9d7 Compare February 26, 2024 15:55
@JordiBForgeFlow
Copy link
Contributor Author

@LoisRForgeFlow @AaronHForgeFlow What do you think about this?

@JordiBForgeFlow JordiBForgeFlow force-pushed the 16.0-fix-rma-get_available_quantity branch from b53a9d7 to db6439f Compare February 26, 2024 15:59
@AaronHForgeFlow
Copy link
Contributor

ok to me. I would make the fields default=True so it behaves the same as it behaved before the changes.

@JordiBForgeFlow JordiBForgeFlow force-pushed the 16.0-fix-rma-get_available_quantity branch 3 times, most recently from 3a19fa4 to a0d0ca5 Compare February 28, 2024 10:14
@JordiBForgeFlow
[IMP] introduce fields in_force_same_lot and out_force_same_lot
a9ac985 
Those fields in the rma.operation allows us to control if we want to ensure that the
same lot as the one indicated in the RMA should be used in deliveries to customers
and receipts from suppliers
@JordiBForgeFlow JordiBForgeFlow force-pushed the 16.0-fix-rma-get_available_quantity branch from a0d0ca5 to a9ac985 Compare May 9, 2024 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JordiBForgeFlow @AaronHForgeFlow