Skip to content

[17.0] [ADD] mail_discuss_channel_hide_sidebar: New module#1828

Open
miquelalzanillas wants to merge 1 commit intoOCA:17.0from
APSL:17.0-add-mail_discuss_channel_hide_sidebar
Open

[17.0] [ADD] mail_discuss_channel_hide_sidebar: New module#1828
miquelalzanillas wants to merge 1 commit intoOCA:17.0from
APSL:17.0-add-mail_discuss_channel_hide_sidebar

Conversation

@miquelalzanillas
Copy link
Copy Markdown

This module adds a "Hide from list" action in Discuss sidebar channels.

The action hides the channel for the current user without unsubscribing.
If a new message arrives in that channel, it appears again automatically.

@mpascuall @BernatObrador @ppyczko

@miquelalzanillas miquelalzanillas force-pushed the 17.0-add-mail_discuss_channel_hide_sidebar branch from 7f29f2d to a526369 Compare March 17, 2026 23:37
Copy link
Copy Markdown

@ppyczko ppyczko left a comment

Choose a reason for hiding this comment

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

LGTM! Tested in runboat.

@BernatObrador
Copy link
Copy Markdown

Works as expected. The issue is that if you hide a channel and later want to send a message in it, you may run into a problem: if no one sends a message after you’ve hidden the channel, you won’t be able to make it visible again.

A possible solution could be to add a menu or section for hidden channels, or something similar.

@miquelalzanillas
Copy link
Copy Markdown
Author

Works as expected. The issue is that if you hide a channel and later want to send a message in it, you may run into a problem: if no one sends a message after you’ve hidden the channel, you won’t be able to make it visible again.

A possible solution could be to add a menu or section for hidden channels, or something similar.

Thanks for the feedback. After hide a channel, you can access to the channel by browsing the channel in the search channel field or if the channel has a new message in the future it will appear again in the list.

@miquelalzanillas
Copy link
Copy Markdown
Author

Hi @hbrunn ,

What do you think about this module? Can we merge it?

Thanks,

Copy link
Copy Markdown

@mpascuall mpascuall left a comment

Choose a reason for hiding this comment

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

LGTM!

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants