Denis Roussel (ACSONE)

Results 1281 comments of Denis Roussel (ACSONE)

/ocabot migration product_contract

@mourad-ehm Could you check pre-commit and attend last comments?

> > @mourad-ehm can you fix pre-commit and test ? > > I did it but the error persist. I fixed it by this commande in local 'pre-commit run --show-diff-on-failure...

@mourad-ehm Do a git status locally. You'll see unadded setup folder. Tested with your branch: ![image](https://user-images.githubusercontent.com/19529533/184873662-ad27b1c9-06d4-48f3-8524-61db5964f552.png)

@mourad-ehm Could you squash not interesting commits ?

> > Thanks @rousseldenis > > Hi @mourad-ehm Thanks for rebase! Can you fix the test ? Which test ?

@etobella Could you answer @kittiu questions ?

@kittiu My question: Why depending on agreement_legal and not just on agreement as far as I can see, no code depends on that one?

> I was testing this fix, but still get the same error message. After archiving all contracts with "Date End" < now (Date of next invoice) I was able to...