-
-
Notifications
You must be signed in to change notification settings - Fork 199
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][IMP] sale_financial_risk_info: groups to risk_info field #381
[16.0][IMP] sale_financial_risk_info: groups to risk_info field #381
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, the objective of this module is for any salesperson with the minimum permissions to see this information, but not have access to the customer risk info tab.
If you want, you can add a new view with a configuration option that activates one or the other, but without changing the default behavior of the module.
You also have the option to customize the system parameter sale_financial_risk_info.info_pattern so that, for example, it only shows the percentage |
I've searched that pattern and it doesn't exist |
Because we use a default value, but you can create this parameter sale_financial_risk_info.info_pattern to be used here: credit-control/sale_financial_risk_info/models/sale.py Lines 20 to 24 in 52e46b3
|
This is explained in readme: |
A user without assigned risk permissions can view the 'Risk Info' field.
Without having User or Manager permissions, you should not see that field.