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

[14.0] [IMP] sale_commission_geo_assign #454

Merged

Conversation

renda-dev
Copy link
Contributor

@renda-dev renda-dev commented Sep 14, 2023

This PR aims to improve some little things in sale_commission_geo_assign.

With the current situation it's not really helpful in a situation like: "Agent A has stopped working with our company, so Agent B must be assigned to Agent A's former customers"

To address those things there are several improvements:

  • Name of the action more clear.
  • New boolean in the wizard to ask if the user wants to overwrite or add agents.
  • New boolean in customer to protect him from being updated by the wizard

@OCA-git-bot
Copy link
Contributor

Hi @eLBati,
some modules you are maintaining are being modified, check this out!

@renda-dev renda-dev changed the title [IMP] sale_commission_geo_assign [14.0] [IMP] sale_commission_geo_assign Sep 15, 2023
@renda-dev renda-dev force-pushed the 14.0-imp-sale_commission_geo_assign branch from 1ee4492 to 9c4cdee Compare September 15, 2023 12:21
Copy link
Member

@ilyasProgrammer ilyasProgrammer left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@francesco-ooops francesco-ooops left a comment

Choose a reason for hiding this comment

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

Functional ok!

This change is both an IMP to features and a clarification of initial behavior (which has been left untouched).

@eLBati good for you?

@aleuffre aleuffre force-pushed the 14.0-imp-sale_commission_geo_assign branch 4 times, most recently from 60aec4d to 46c519a Compare September 19, 2023 12:25
@aleuffre aleuffre force-pushed the 14.0-imp-sale_commission_geo_assign branch from 46c519a to f925c46 Compare September 19, 2023 12:27
@francesco-ooops
Copy link
Contributor

@pedrobaeza can we merge?

@pedrobaeza pedrobaeza added this to the 14.0 milestone Sep 19, 2023
@pedrobaeza
Copy link
Member

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-454-by-pedrobaeza-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 36db5b1 into OCA:14.0 Sep 19, 2023
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at a1fc50a. Thanks a lot for contributing to OCA. ❤️

@aleuffre aleuffre deleted the 14.0-imp-sale_commission_geo_assign branch September 19, 2023 14:14
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.

5 participants