web icon indicating copy to clipboard operation
web copied to clipboard

[FP][17.0][WIP] [IMP] web_timeline - support multiple group-by levels and allow to group by m2m field

Open cuongnmtm opened this issue 3 months ago • 3 comments

Test cases:

  • [ ] group by one field (group by m2o, selection, integer, m2m, date, ...)
  • [ ] group by multiple fields
  • [ ] group by 0 field: back to default group by
  • [ ] create a new record
  • [ ] drag and drop record to update
  • [ ] click on the group by to view the group by record detail
  • [ ] when group by m2m field, record is displayed multiple times if linked to multiple m2m records
  • [ ] when drag and drop, the value of the m2m field should not be updated as there is no best way to handle this
  • [ ] when creating a new record, set the default value for all grouped fields
  • [ ] handle group by non-relation fields: click on the group of selection or char field do not popup record form view to avoid error.
  • [ ] dependency_arrow between multiple collapse/expand groups

cuongnmtm avatar Oct 02 '25 08:10 cuongnmtm

Hi @tarteo, some modules you are maintaining are being modified, check this out!

OCA-git-bot avatar Oct 02 '25 08:10 OCA-git-bot

@cuongnmtm I understand this PR is not ready for review yet because the files are not loaded in the manifest. Additionally, you must convert them to .esm.js and odoo-module according to this commit made during the migration to 17.0: https://github.com/OCA/web/commit/43341291dd0a61875fb4e45e75baff096420ac5e

carlos-lopez-tecnativa avatar Oct 03 '25 12:10 carlos-lopez-tecnativa

I got this error when grouping a datetime field. I suspect this commit 751fb9c might be missing, please review:

image

I haven't finished this port yet. There are three commits to port, and the remaining two are taking more time because I need to adapt the changes to the refactoring in version 17. I will find time to work on them.

cuongnmtm avatar Oct 09 '25 17:10 cuongnmtm