Skip to content

[16.0] [IMP] announcement: allow exclude users/groups#1235

Merged
OCA-git-bot merged 2 commits intoOCA:16.0from
dixmit:16.0-imp-announcement
Feb 17, 2026
Merged

[16.0] [IMP] announcement: allow exclude users/groups#1235
OCA-git-bot merged 2 commits intoOCA:16.0from
dixmit:16.0-imp-announcement

Conversation

@luisDIXMIT
Copy link
Contributor

With this change, we also allow excluding specific users or groups from the user list. A page has been added to clearly show the users’ scope.

Imagine a large company with many users. They want to send a generic announcement but also need to exclude certain users. Furthermore, they may want to exclude several users who belong to a specific group.

This way, they can manage a large number of users effortlessly.

@etobella

@luisDIXMIT
Copy link
Contributor Author

If you see correct the change, I also can update the readme to explain the new functionality.

@pedrobaeza pedrobaeza added this to the 16.0 milestone Feb 4, 2026
@pedrobaeza
Copy link
Member

Yes, I think it's a valid improvement of the module to complement the existing things.

@luisDIXMIT luisDIXMIT force-pushed the 16.0-imp-announcement branch 3 times, most recently from 278c2dc to 0fe93be Compare February 4, 2026 14:28
@luisDIXMIT
Copy link
Contributor Author

The test error is about connection failed.

@luisDIXMIT luisDIXMIT force-pushed the 16.0-imp-announcement branch from 0fe93be to 879d989 Compare February 17, 2026 09:22
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-1235-by-pedrobaeza-bump-minor, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Feb 17, 2026
Signed-off-by pedrobaeza
@OCA-git-bot
Copy link
Contributor

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-1235-by-pedrobaeza-bump-minor.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@pedrobaeza
Copy link
Member

This is affected by the setuptools problem. This requires OCA/oca-addons-repo-template#333 or to switch to whool.

@OCA-git-bot
Copy link
Contributor

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-1235-by-pedrobaeza-bump-minor.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@luisDIXMIT
Copy link
Contributor Author

@pedrobaeza Is valid the solution I made?

@pedrobaeza
Copy link
Member

Let's try...

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-1235-by-pedrobaeza-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 2abfd88 into OCA:16.0 Feb 17, 2026
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at bd1818f. Thanks a lot for contributing to OCA. ❤️

@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). 🤖

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.

4 participants