Skip to content

Conversation

@faizanlodhi
Copy link

Summary

This PR adds a new module dependency_map_view, which introduces a new view type allowing users to visualize dependencies between records in a graph or map layout.

Features

  • Displays relationships between models/records.
  • Helps track record-level dependencies visually.
  • Integrates seamlessly with existing Odoo models.

Usage

Once installed, users can open the Dependency Map view from the corresponding model views to explore related data.

Task/Issue

N/A (New feature proposal)

@legalsylvain
Copy link
Contributor

Hi @faizanlodhi. Thanks for sharing this module.
I made some tests on runboat, and I don't understand the use case of this feature.
image

could you elaborate in the USAGE.md file, and add some screenshot ?

thanks !

(also, could you squash your commits ?)

Copy link
Contributor

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

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

Hi. I Guess you used AI to generate the code ? As a result, there is a lot of totally useless comments. Could you clean that ?

thanks in advance.

@@ -0,0 +1,5 @@
*.pyc
Copy link
Contributor

Choose a reason for hiding this comment

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

useless file to remove.


def post_init_hook(env):
"""
Add dependency_map view mode to all existing window actions.
Copy link
Contributor

Choose a reason for hiding this comment

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

I see it more as an optional action. I don't think all people want to use this new view in all models.

Comment on lines 8 to 16
Modules:
- ir_ui_view: Extends ir.ui.view to add 'dependency_map' as a new view type
- ir_actions: Extends ir.actions.act_window to automatically add dependency
map view to actions for models with relational fields


"""

# Import view model extension
Copy link
Contributor

Choose a reason for hiding this comment

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

all comments can be removeds.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants