-
-
Notifications
You must be signed in to change notification settings - Fork 871
[18.0][ADD] project_sale_link #1583
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
base: 18.0
Are you sure you want to change the base?
Conversation
mathben
commented
Oct 15, 2025
- copying project_purchase_link and rename purchase to sale
5fc10a4 to
433c0b2
Compare
- copying project_purchase_link and rename purchase to sale
0d27c75 to
3310ff0
Compare
4f2c2e7 to
a5c3038
Compare
d209d40 to
2ed9dcb
Compare
…ulation - use move_out for sale - count invoice use move_id_count - count sale use price_subtotal_count - detection analytic_distribution change add_where request - improve test
2ed9dcb to
a68adfc
Compare
flotho
left a comment
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.
Hi,
LGTM
sucessfully tested on runboat
|
ping @carlos-lopez-tecnativa @pedrobaeza |
|
Copy from where? Did the previous authors sign the OCA CLA? You should also respect the whole commit history, and rename it from the beginning with: |
This repository : https://github.com/OCA/project/tree/18.0/project_purchase_link
I didn't check : https://github.com/OCA/project/blob/18.0/project_purchase_link/readme/CONTRIBUTORS.md
The history : https://github.com/OCA/project/commits/18.0/project_purchase_link |
|
OK, then it's not a copy, but it's a "brother" module inspired from the other. Isn't this what |
|
Hello, we formerly already had a similar module in 13.0 -> #940 |