Skip to content
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

18.0 [ADD] web_hierarchy_list #2970

Open
wants to merge 1 commit into
base: 18.0
Choose a base branch
from

Conversation

IT-Ideas
Copy link

@IT-Ideas IT-Ideas commented Oct 22, 2024

This PR is about adding a readonly hierarchy view in odoo based on the used of the existing list view and by adding an embedded breadcrumb representing the parents path during the exploration.

Remaining Improvements to be done in a later roadmap:

  • Prevent fetching all child_ids by adding the field name option in the parent field and performing a read_group for the ids that are actually rendered.
  • Implement CRUD in one2many widget.
image image

@pedrobaeza
Copy link
Member

Any illustrating screenshot?

@IT-Ideas
Copy link
Author

IT-Ideas commented Oct 22, 2024

Any illustrating screenshot?

Here you go!

@pedrobaeza pedrobaeza added this to the 18.0 milestone Oct 22, 2024
@IT-Ideas
Copy link
Author

Hey @aab-odoo, @ged-odoo

If you would have a minute for me and have a look at the concept I would be very happy to get your feedback.
Proceeding this way (with an embedded breadcrumb) removes the issues we had foreseen regarding the Filters and Search. Indeed this way you always get the result corresponding to your criteria and then you can start your drill downs (keeping you path at sight)

@IT-Ideas
Copy link
Author

@lmignon, @sbidoul,

Here is already the very very draft PR 🙂

@IT-Ideas IT-Ideas force-pushed the 18.0-hierachy_tree-lst branch 2 times, most recently from 7cc2d9d to 947746a Compare October 23, 2024 09:21
@ged-odoo
Copy link

just had a quick look. x2manys are hard, so it's quite impressive you managed to do that, good job. From a technical point of view, looks very good. Nothing really stands out as a problem, code seems well organized.

Functionally, it makes sense to me, but i am not an expert. (also, note that odoo will likely work on a tree view in the future, not sure how much it will overlap this, but i guess it's good to know)

@IT-Ideas
Copy link
Author

just had a quick look. x2manys are hard, so it's quite impressive you managed to do that, good job. From a technical point of view, looks very good. Nothing really stands out as a problem, code seems well organized.

Functionally, it makes sense to me, but i am not an expert. (also, note that odoo will likely work on a tree view in the future, not sure how much it will overlap this, but i guess it's good to know)

Hey @ged-odoo thanks so much for having taken the time to have a look 🙂. I owe you some beers 😉 I'll pass by the farm soon to hang out with the Vidange / Services team. I did not get too much time to share with you guys at the OXP.

@sbidoul
Copy link
Member

sbidoul commented Oct 23, 2024

🏅 @IT-Ideas :)

@IT-Ideas IT-Ideas force-pushed the 18.0-hierachy_tree-lst branch 5 times, most recently from 3367f1d to af7d044 Compare October 30, 2024 14:07
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.

4 participants