[18.0] [FIX] mis_builder: account.account multi company support for Odoo 18#756
[18.0] [FIX] mis_builder: account.account multi company support for Odoo 18#756
Conversation
|
Hi @sbidoul, |
f889173 to
995020c
Compare
Co-authored-by: Stefan Rijnhart <[email protected]> Co-authored-by: Geraldo Lopez <[email protected]>
995020c to
4100527
Compare
|
Just added translations to "and <nbr_companies_exceeded> more" and translations for other missing terms in es_ES. |
|
This PR has the |
sbidoul
left a comment
There was a problem hiding this comment.
Thanks!
The PR is fixes the issue but introduces a change, in the sense that before, there was one detail line per account and company.
I think I more or less know what to do with a first baby step towards configurable account detail expansion.
Marking as request change for now as I'm not convinced yet this is a good default behaviour.
I hope to come to this soon.
|
IMHO the fix should be merged and, later, add that configurability. Otherwise the module explodes in users' hands right now. |
|
@Shide could we apply this PR in PROD? The issue is affecting production clients enviroments. Thank you! |
|
@rafaelbn, unless @sbidoul are making another PR that makes this case better structured, I think we could. @sbidoul, could you ping us to know when the new PR will replace this one? Thanks |
Adapted from #711 and #730.
Limit is set con code and now query_companies it's used to show properly the companies that are being queried.
Also fixed the account code that shows to False if the env.company is not the fetched company data.
Tests are mainly from @StefanRijnhart, but adapted a little bit.
Coauthored with @StefanRijnhart and @geraldo29
MT-11000 @moduon @rafaelbn @StefanRijnhart @geraldo29 @sbidoul @ThiagoMForgeFlow please review if you want 😄