Skip to content
Snippets Groups Projects

Added role_coordination and role_internal_user roles

Merged Benjamí Ramos requested to merge feature/coordinator_and_user_roles into dev
1 unresolved thread
  • role_coordination: For Coortinator companies :sparkles:
  • role_internal_user: Required for Odoo login and access :cartwheel:

Merge request reports

Pipeline #33265 passed

Pipeline passed for fe0e66a5 on feature/coordinator_and_user_roles

Approved by

Merged by Benjamí               RamosBenjamí Ramos 1 year ago (Jun 26, 2023 10:08am UTC)

Merge details

  • Changes merged into dev with 13fcaae8.
  • Did not delete the source branch.

Pipeline #33876 passed

Pipeline passed for 13fcaae8 on dev

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
35 35 <record model="res.users.role" id="role_ce_admin">
36 36 <field name="name">Energy Community Administrator</field>
37 37 <field name="code">role_ce_admin</field>
38 <field name="implied_ids"
39 eval="[
38 <field name="implied_ids" eval="[
39 (4, ref('group_admin')),
40 (4, ref('sale.group_delivery_invoice_address')),
41 (4, ref('account.group_account_invoice')),
42 (4, ref('base.group_partner_manager')),
43 (4, ref('base.group_user')),
  • We talked about this today, to remove a bug from the roles assignation we need to have the minimum "user type" group assigned without a company.

    There are 3 (public, portal and internal user) user type groups, and if we use deferents for some roles (like a CE member that is just a portal user), we need to have one for every base "user type" and assign them without explicit company.

  • Benjamí Ramos added 1 commit

    added 1 commit

    • a8131e38 - Removed base.group_user from non base roles

    Compare with previous version

  • Maybe we should remove the base.group_portal from the member role too and have a base role for portal user assigned without company to the CE members.

    https://git.coopdevs.org/coopdevs/comunitats-energetiques/odoo-ce/-/blob/36d97574c09d00f95e9f2331937fa9e50f66760d/energy_communities/security/res_users_role_data.xml#L32

    Because it should cause the same bug as the internal users, but we should test it.

  • Benjamí Ramos added 1 commit

    added 1 commit

    • fe0e66a5 - Added coordinator group and remove portal group from role_ce_member

    Compare with previous version

  • Enrico Stano approved this merge request

    approved this merge request

  • Xavier Bonet approved this merge request

    approved this merge request

  • Benjamí Ramos mentioned in commit 13fcaae8

    mentioned in commit 13fcaae8

  • merged

  • mentioned in commit 6e105c32

  • mentioned in merge request !149 (merged)

  • mentioned in commit f06a1a6d

  • requested review from @DaniilDigtyar

  • assigned to @benjami

  • Benjamí Ramos mentioned in merge request !157 (closed)

    mentioned in merge request !157 (closed)

  • Benjamí Ramos mentioned in commit a80baefa

    mentioned in commit a80baefa

  • Benjamí Ramos mentioned in merge request !158 (merged)

    mentioned in merge request !158 (merged)

  • Benjamí Ramos mentioned in commit dd5b8700

    mentioned in commit dd5b8700

  • Please register or sign in to reply
    Loading