+
Skip to content

Conversation

cuongnmtm
Copy link
Contributor

@cuongnmtm cuongnmtm commented Oct 2, 2025

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

@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza added this to the 17.0 milestone Oct 2, 2025
@carlos-lopez-tecnativa
Copy link
Contributor

@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: 4334129

@cuongnmtm cuongnmtm force-pushed the web_timeline-multi-group-by-fp-17 branch from 67df4c0 to 62b9a52 Compare October 5, 2025 16:53
Copy link
Contributor

@carlos-lopez-tecnativa carlos-lopez-tecnativa left a comment

Choose a reason for hiding this comment

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

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

image

@cuongnmtm
Copy link
Contributor Author

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.

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

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载