Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0][ADD] sale_loyalty_industry #197

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

santostelmo
Copy link

This module allow to restrict the industries in which the sale loyalty program applies.

Note: Moved from sale-workflow OCA/sale-workflow#2859

Copy link

@cyrilmanuel cyrilmanuel left a comment

Choose a reason for hiding this comment

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

LGTM

@cyrilmanuel
Copy link

Hi @pedrobaeza can you make a review here please ?

@pedrobaeza pedrobaeza added this to the 16.0 milestone Apr 13, 2024
@pedrobaeza pedrobaeza changed the title [ADD] sale_loyalty_industry [16.0][ADD] sale_loyalty_industry Apr 13, 2024
@pedrobaeza
Copy link
Member

I think this module is not needed if you use loyalty_partner_applicability (and its sales counterpart sale_loyalty_partner_applicability, which is more generic to filter partners by any condition (including the industry), and it's also done at rule level, not program level, which is more inline with the new v16 approach.

imagen

I know yours is more "direct", but it's very limiting and in the end, I think using the generic one will give more versatility to your customer.

@santostelmo
Copy link
Author

I think this module is not needed if you use loyalty_partner_applicability (and its sales counterpart sale_loyalty_partner_applicability, which is more generic to filter partners by any condition (including the industry), and it's also done at rule level, not program level, which is more inline with the new v16 approach.

imagen

I know yours is more "direct", but it's very limiting and in the end, I think using the generic one will give more versatility to your customer.

I have tested this module sale_loyalty_partner_applicability and indeed it seems to cover the needs. But i'm afraid that this domain is hard for a odoo user to build. It's less user friendly.

Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Nov 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants