Skip to content

[18.0][ADD] sale_invoice_product_not_alone: add new module#4366

Open
eduezerouali-tecnativa wants to merge 1 commit into
OCA:18.0from
Tecnativa:18.0-add-product_invoice_policy_not_alone
Open

[18.0][ADD] sale_invoice_product_not_alone: add new module#4366
eduezerouali-tecnativa wants to merge 1 commit into
OCA:18.0from
Tecnativa:18.0-add-product_invoice_policy_not_alone

Conversation

@eduezerouali-tecnativa
Copy link
Copy Markdown

cc @Tecnativa TT62537
ping @sergio-teruel @carlosdauden

This module allows to set products that can't be invoice alone. This makes the sale not to appear on "Orders to invoice".

@OCA-git-bot OCA-git-bot added series:18.0 mod:product_not_invoice_alone Module product_not_invoice_alone labels May 26, 2026
@eduezerouali-tecnativa eduezerouali-tecnativa force-pushed the 18.0-add-product_invoice_policy_not_alone branch from 52daddd to 9ee3ddf Compare May 26, 2026 12:58
@pedrobaeza
Copy link
Copy Markdown
Member

The module name should start with sale_. For example, sale_invoicing_product_not_alone.

@eduezerouali-tecnativa eduezerouali-tecnativa force-pushed the 18.0-add-product_invoice_policy_not_alone branch from 9ee3ddf to 923e2e3 Compare May 27, 2026 07:47
@OCA-git-bot OCA-git-bot added mod:sale_invoice_product_not_invoice_alone Module sale_invoice_product_not_invoice_alone and removed mod:product_not_invoice_alone Module product_not_invoice_alone labels May 27, 2026
@eduezerouali-tecnativa eduezerouali-tecnativa changed the title [18.0][ADD] product_not_invoice_alone: add new module [18.0][ADD] sale_invoice_product_not_invoice_alone: add new module May 27, 2026
@eduezerouali-tecnativa
Copy link
Copy Markdown
Author

module was rename to sale_invoice_product_not_alone

@eduezerouali-tecnativa eduezerouali-tecnativa force-pushed the 18.0-add-product_invoice_policy_not_alone branch from 923e2e3 to 695b108 Compare May 27, 2026 08:09
@OCA-git-bot OCA-git-bot added mod:sale_invoice_product_not_alone Module sale_invoice_product_not_alone and removed mod:sale_invoice_product_not_invoice_alone Module sale_invoice_product_not_invoice_alone labels May 27, 2026
@eduezerouali-tecnativa eduezerouali-tecnativa force-pushed the 18.0-add-product_invoice_policy_not_alone branch from 695b108 to 53ad55d Compare May 27, 2026 11:33
@@ -0,0 +1,2 @@
This module allows to set on product if it can be invoice alone. This make those sale order not to appear
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
This module allows to set on product if it can be invoice alone. This make those sale order not to appear
This module allows to set on product if it can be invoiced alone. This makes those sale orders not to appear

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. Thanks for your review!!

@eduezerouali-tecnativa eduezerouali-tecnativa force-pushed the 18.0-add-product_invoice_policy_not_alone branch from 53ad55d to 7cb92f3 Compare May 27, 2026 12:59
@eduezerouali-tecnativa eduezerouali-tecnativa changed the title [18.0][ADD] sale_invoice_product_not_invoice_alone: add new module [18.0][ADD] sale_invoice_product_not_alone: add new module May 27, 2026
@eduezerouali-tecnativa eduezerouali-tecnativa force-pushed the 18.0-add-product_invoice_policy_not_alone branch from 7cb92f3 to de2d589 Compare May 27, 2026 13:00
@eduezerouali-tecnativa eduezerouali-tecnativa force-pushed the 18.0-add-product_invoice_policy_not_alone branch from de2d589 to 184604b Compare May 27, 2026 15:54
Copy link
Copy Markdown
Contributor

@sergio-teruel sergio-teruel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Copy link
Copy Markdown
Contributor

@BhaveshHeliconia BhaveshHeliconia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@sergio-teruel
Copy link
Copy Markdown
Contributor

Hi @rousseldenis Can you trigger the merge??

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants