Thierry Ducrest

@grindtildeath Could you squash and see if the pre-commit still have an issue, thanks

@grindtildeath CI is red :crying_cat_face:

Thanks @grindtildeath CI is :green_heart: again Squash and merge next ?

> no need to update tests? I have added 2 in relation to the changes

> LG :) Can you squash fixups? Done

> @TDu I have two questions on this as it seems to have not been integrated in OCA module: > > * The grouping mode and the invoicing mode on...

> I find it strange to return ids and not recordset True, I did not want to browse for the records when not needed.

> @TDu Prefer explicit naming `sale_stock` in commit message: > >  I updated the commit message. Always a juggle :juggling_person: between the length and the meaning on the title...

> > I updated the commit message. Always a juggle 🤹 between the length and the meaning on the title of the commit message. > > But the module name...

> You squash and update description? Done