Skip to content
Snippets Groups Projects

[REM] energy_project_energy_communities and energy_selfconsumption_cooperator

Merged Daniil Digtyar Vasilieva requested to merge REM_energy_project_energy_communities into dev
1 unresolved thread

Explanation for energy_project_energy_communities module

With the last !210 (merged) MR the energy_project_energy_communities module only was used to add the groups to the roles declared in energy_communities.

The module energy_project_energy_communities was first added to not add a dependency in energy_communities with all the energy project related development. Now that doesn't make sense to our case because energy_communities is specifically a module developed to solve the specific cases of Som Comunitats project and needs the energy_project modules as it is in already in his dependencies (commit), so we can generate the demo data needed.

Explanation for energy_selfconsumption_cooperator module

Similar case, but so we don't generate a dependency between energy_selfconsumption and cooperator. We wanted to be able energy_selfconsumption to not depend on the coopeartor logic for the partners related, mostly in the inscriptions. But this added a lot of complexity with the last (commit) change, and it doesn't make sense to add complexity for a feature that we may not use. Still, we will document #38 all the dependencies uses to cooperator so if someone wants to remove it will be easier.

If we remove these modules, we remove the complexity related to the deployment and dependency structure of this project.

Edited by Daniil Digtyar Vasilieva

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • 24 24 )
    25 25 partner_id = fields.Many2one(
    26 26 "res.partner",
    27 string="Partner",
    27 string="Cooperator",
  • added 1 commit

    • 634c17df - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • added 1 commit

    • 73333d76 - [REF] energy_communities: renamed method to reflect partner_id

    Compare with previous version

  • added 1 commit

    • 551c1e13 - [CLN] energy_communities: format

    Compare with previous version

  • added ToBeMerged label and removed ToBeReviewed label

  • mentioned in commit fafad2cd

  • mentioned in merge request !220 (merged)

  • Please register or sign in to reply
    Loading