Skip to content

[17.0][ADD] mail_chatter_split#1776

Closed
AndreuOForgeFlow wants to merge 1 commit intoOCA:17.0from
ForgeFlow:17.0-add-mail_chatter_split
Closed

[17.0][ADD] mail_chatter_split#1776
AndreuOForgeFlow wants to merge 1 commit intoOCA:17.0from
ForgeFlow:17.0-add-mail_chatter_split

Conversation

@AndreuOForgeFlow
Copy link

@AndreuOForgeFlow AndreuOForgeFlow commented Dec 11, 2025

This module introduces filter buttons in the chatter to separate user messages from system logs.
It adds a small toggle toolbar above the chatter that allows switching between:

  • All messages: default view showing all messages
    image

  • User messages: only user-generated messages
    image

  • System logs: only system-generated logs
    image

  • Planned activities:
    image

@AndreuOForgeFlow AndreuOForgeFlow marked this pull request as ready for review December 11, 2025 09:47
@AndreuOForgeFlow AndreuOForgeFlow force-pushed the 17.0-add-mail_chatter_split branch from b7fad71 to 2e74ae8 Compare December 11, 2025 14:14
Copy link

@GuillemCForgeFlow GuillemCForgeFlow left a comment

Choose a reason for hiding this comment

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

really interesting feature IHMO, functional and code review 👍🏿

@AndreuOForgeFlow AndreuOForgeFlow force-pushed the 17.0-add-mail_chatter_split branch 2 times, most recently from e78fe01 to e87867b Compare December 16, 2025 12:22
@AndreuOForgeFlow AndreuOForgeFlow force-pushed the 17.0-add-mail_chatter_split branch 2 times, most recently from 9be8865 to 02d57c5 Compare December 18, 2025 16:24
@AndreuOForgeFlow AndreuOForgeFlow force-pushed the 17.0-add-mail_chatter_split branch from 02d57c5 to 4bb9015 Compare December 19, 2025 10:00
Copy link

@hoangtrann hoangtrann left a comment

Choose a reason for hiding this comment

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

Code LGTM

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@pedrobaeza
Copy link
Member

As this is a new module, it can go directly to the 17.0 branch of OCA/mail.

Copy link
Member

@etobella etobella left a comment

Choose a reason for hiding this comment

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

I agree with @pedrobaeza , this should go to mail, also, can you review the responsive view (try to see how it is seen with a mobile interface)

Image

IMO, in mobile, you should remove the text.

@AndreuOForgeFlow
Copy link
Author

Moving this to OCA/mail:

Closing this PR

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants