-
-
Notifications
You must be signed in to change notification settings - Fork 796
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
[17.0][ADD] sale_project_reimbursement_cost: New module to display provisions and reimbur… #1388
base: 17.0
Are you sure you want to change the base?
[17.0][ADD] sale_project_reimbursement_cost: New module to display provisions and reimbur… #1388
Conversation
On http://oca-project-17-0-pr1388-5e093d29ef25.runboat.odoo-community.org/web#id=24&cids=1&menu_id=260&action=424&model=sale.order&view_type=form, check that the line with the provision is not added to the corresponding section in the "To invoice". |
@pedrobaeza According to the requirements, the |
OK, understood. My bad. Then what you have to do is to not put such columns there. Only the column of "effective" amounts (the "Invoiced" one). |
OK, it's correct. I saw the other 3 columns, but again I should see badly, as now I'm not able to reproduce it. I think it's because you have already changed some things in the runboat. But what about the reimbursable cost that is already invoiced and should be deducted from that section? |
5e093d2
to
f5558c8
Compare
@pedrobaeza I updated the code to display only a table with provision lines and reimbursable lines. Additionally, I formatted the date. Please update your review. |
I get this error in runboat:
Also you can add product demo data for not having to set all the master data each time. |
2ec106e
to
95be91f
Compare
95be91f
to
2e12640
Compare
…sement costs in the Project Updates dashboard
2e12640
to
f4c46c1
Compare
@pedrobaeza According to the new flow, when validating the bill, if the provision has a remaining amount, a new line is added to the sale order with a negative quantity. Please update your review. Thanks in advance. |
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.
Tested the new flow and working.
…sement costs in the Project Updates dashboard
TT52049
@Tecnativa
@pedrobaeza could you please review this