-
-
Notifications
You must be signed in to change notification settings - Fork 289
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
[16.0][ADD] sale_purchase_inter_company #456
[16.0][ADD] sale_purchase_inter_company #456
Conversation
48e0b48
to
66d9170
Compare
66d9170
to
690adca
Compare
@sebastienbeau , @Kev-Roche , can you review, please ? |
Does this module plan to be compatible with the stock type product as well ? I tried this on runboat. I installed the Inventory to test as a stock type Product.
|
This module is compatible with the service type product i.e. when you do not want to depend on the Inventory module. |
Please add an unit test to run the uncovered function mentioned by codecov |
If you don't mind, I'll add it |
Yes, I do. Thank you for your time. |
690adca
to
df2f5b0
Compare
Any review @Kev-Roche, @sebastienbeau , @metaminux ? |
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
Hello @chafique-delli , I made a fix here, what do you think ? akretion#14 |
cc @sebastienbeau , @Kev-Roche