OCA / multi-company

GNU Affero General Public License v3.0
100 stars 281 forks source link

[16.0][ADD] base_company_legal_info #674

Open quentinDupont opened 1 month ago

quentinDupont commented 1 month ago

Custom module used for several years in production. Migration in v16.

This module adds legal fields on companies :

With that design, you can now use the name field for the Trade name of your companies.

res_company_form report_layout_footer

rvalyi commented 1 month ago

Hello @quentinDupont I wonder if it's really multi-companies oriented to belong to this repo. Eventually it could belong to https://github.com/OCA/partner-contact instead no? Just like thecompany modules of this partner_contact repo, no?

Also for the legal name there is already a many2one res.partner.company.type field, see https://github.com/OCA/partner-contact/blob/16.0/partner_company_type/models/res_partner.py#L11 Is a new text field necessary? Just asking.

quentinDupont commented 1 month ago

Hello @quentinDupont I wonder if it's really multi-companies oriented to belong to this repo. Eventually it could belong to https://github.com/OCA/partner-contact instead no? Just like thecompany modules of this partner_contact repo, no?

Also for the legal name there is already a many2one res.partner.company.type field, see https://github.com/OCA/partner-contact/blob/16.0/partner_company_type/models/res_partner.py#L11 Is a new text field necessary? Just asking.

ooooh res.partner.company.type is very interesting thanks ! Indeed i was not sure for the repo and seeing partner_company_type/ in this repo make me think ! But it is really related to company and not partner

legalsylvain commented 1 month ago

Hello @quentinDupont I wonder if it's really multi-companies oriented to belong to this repo. Eventually it could belong to https://github.com/OCA/partner-contact instead no? Just like thecompany modules of this partner_contact repo, no?

Hi. could you elaborate I don't see module related to "res.company" model in this repo. I see modules related to partner "company = True" but it is unrelated. Or do you mean we should add that fields on res.partner level ?

rvalyi commented 1 month ago

Hello @legalsylvain, may be these modules from partner-contact aren't company oriented. Still this doesn't make this new module fit a "multi" company purpose so I still wonder if it the proper repo... Or eventually we might decide that this repo is more about res.company extension in general than simply multi-companies. What do you think?

legalsylvain commented 1 month ago

Or eventually we might decide that this repo is more about res.company extension in general than simply multi-companies. What do you think?

I get your point. Well, In fact it's a multi-company module by design. In a mono-company context, if you want to change the report, you can edit manually the report template and add custom text. but if you are in a multi company like in our instance (> 70 companies) you can't add custom fixed text. So this module begin to be interested.

What do you think ?

rvalyi commented 1 month ago

Or eventually we might decide that this repo is more about res.company extension in general than simply multi-companies. What do you think?

I get your point. Well, In fact it's a multi-company module by design. In a mono-company context, if you want to change the report, you can edit manually the report template and add custom text. but if you are in a multi company like in our instance (> 70 companies) you can't add custom fixed text. So this module begin to be interested.

What do you think ?

Point taken. I agree it makes sense in the repo then.

flotho commented 1 week ago

ready for review ?

legalsylvain commented 1 week ago

ready for review ?

indeed ! regarding @rvalyi answer, I think that it's the correct repo, so feel free to test & review !