Closed edy1192 closed 4 years ago
@luistorresm Could you check this, please?
Regards.
@JulioSerna @hbto
LGTM this module improve, what do you think?
Regards
An extra LGTM :+1:
@hbto Could you check this, please?
Regards.
:+1:
@moylop260
Could you merge this PR, please?
Regards
@hugho-ad @moylop260 I applied the changes that you suggested! Could you check it please?
Regards.
LGTM :+1:
Hi @moylop260 Could you review / merge this, please?
cc @hugho-ad
Regards
@moylop260 I applied the comments that you gave me in the meeting.
I will ask the team, customers with this module to make dummy MRs and test these changes.
Thanks for your advices.
@hugho-ad @luistorresm Thanks you too
😄
Regards.
For instances of multiple companies, the credit limit value or days of grace does not differentiate between company, so if the customer has a credit limit
100
withcompany A
, that same value will be in thecompany B
.In some cases these values changes according to the company.
With this change the
account.move.line
are searched according to the company and additionally the credit limit fields and grace days now depend on the company.Migration script to set the old values for the
credit_limit
andgrace_payment_days
fields.Videos in the comments: