OCA / multi-company

GNU Affero General Public License v3.0
98 stars 280 forks source link

[17.0][MIG] account_invoice_inter_company #595

Open cuongnmtm opened 4 months ago

cuongnmtm commented 4 months ago

Note:

The test test_confirm_invoice_with_product_and_shared_catalog is failed. The reason is that Odoo v17 now has a check_company=True attribute on the product_id field of the account.move.line, which was not present in v16.

Although the "Common Product Catalog" feature of this module seems appealing, in order to make it work, we need to bypass the Odoo check_company attribute. This makes the feature less appealing and we should consider removing it from the module. Instead, users can remove the company value on some products to make them shared products, which is a normal procedure for inter-company.

=> I will remove the "Common Product Catalog" option.

cuongnmtm commented 4 months ago

The test test_confirm_invoice_with_product_and_shared_catalog is failed. The reason is that Odoo v17 now has a check_company=True attribute on the product_id field of the account.move.line, which was not present in v16.

Although the "Common Product Catalog" feature of this module seems appealing, in order to make it work, we need to bypass the Odoo check_company attribute. This makes the feature less appealing and we should consider removing it from the module. Instead, users can remove the company value on some products to make them shared products, which is a normal procedure for inter-company.