- Sponsor
-
Notifications
You must be signed in to change notification settings - Fork 4
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
[15.0][ADD] website_sale_slides_order_line_link: Add new module #8
[15.0][ADD] website_sale_slides_order_line_link: Add new module #8
Conversation
87ac559
to
39522d7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code review: This module does not add any views or filters. Is it intended as a glue module for another purpose? Could you provide more information or context to better understand its function?
Is indeed a technical module, splitting pulling a functionality from this other work #6 that might be common for other future modules |
39522d7
to
a75f612
Compare
I have added the information to the module to make it a little less confusing. |
a75f612
to
97cbb43
Compare
TT54986
97cbb43
to
f501e10
Compare
This PR has the |
/ocabot merge nobump |
On my way to merge this fine PR! |
Congratulations, your PR was merged at 05d66d9. Thanks a lot for contributing to OCA. ❤️ |
cc @Tecnativa TT54986
@chienandalu @carlos-lopez-tecnativa please review