-
-
Notifications
You must be signed in to change notification settings - Fork 406
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
[12.0][MIG] Backport from 14.0 to 12.0 crm_security_group #597
base: 12.0
Are you sure you want to change the base?
Conversation
…ups (Own Documents Only, All Documents and Administrator) to separate Sales and CRM apps
- Show CRM root menu only if user has any CRM permissions. - Show Reporting menu from CRM only if user has any CRM permissions. TT40356
Currently translated at 100.0% (4 of 4 strings) Translation: crm-14.0/crm-14.0-crm_security_group Translate-URL: https://translation.odoo-community.org/projects/crm-14-0/crm-14-0-crm_security_group/es/
Currently translated at 100.0% (4 of 4 strings) Translation: crm-14.0/crm-14.0-crm_security_group Translate-URL: https://translation.odoo-community.org/projects/crm-14-0/crm-14-0-crm_security_group/it/
94e9bd5
to
95a2474
Compare
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.
hi ! thanks for porting this module ! a blocking remark inline.
access_crm_tag_own_leads,sale_team.crm.tag.own.leads,sales_team.model_crm_tag,crm_security_group.group_crm_own_leads,1,1,1,0 | ||
access_crm_tag_manager,sale_team.crm.tag.manager,sales_team.model_crm_tag,crm_security_group.group_crm_manager,1,1,1,1 | ||
access_crm_lead_scoring_frequency_user,crm.lead.scoring.frequency.user,crm.model_crm_lead_scoring_frequency,crm_security_group.group_crm_own_leads,1,0,0,0 | ||
access_crm_lead_scoring_frequency_field_user,crm.lead.scoring.frequency.field.user,crm.model_crm_lead_scoring_frequency_field,crm_security_group.group_crm_own_leads,1,0,0,0 |
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.
a reason to remove that lines ?
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.
Yep ! Theses fields were created post version 12 :)
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.
thansk !
95a2474
to
0e5b6db
Compare
CI is red in this repo at the moment 😢 |
Changes from 14.0 in this commit :
adjust permissions rights for other module (crm_phonecall) (the right changes was made in migration 14.0 here)Thanks for @victoralmau for all the work, very useful for us !