Skip to content

[ADD] mass_mailing_list_prune_blacklisted#1853

Open
SirPyTech wants to merge 1 commit intoOCA:16.0from
PyTech-SRL:16.0-add-mass_mailing_list_cleanup
Open

[ADD] mass_mailing_list_prune_blacklisted#1853
SirPyTech wants to merge 1 commit intoOCA:16.0from
PyTech-SRL:16.0-add-mass_mailing_list_cleanup

Conversation

@SirPyTech
Copy link
Copy Markdown

Allow the user to massively remove blacklisted contacts from mailing lists.

@OCA-git-bot OCA-git-bot added mod:mass_mailing_list_prune_blacklisted Module mass_mailing_list_prune_blacklisted series:16.0 labels Apr 16, 2026
@SirPyTech SirPyTech force-pushed the 16.0-add-mass_mailing_list_cleanup branch from 0ff03a9 to 266e35d Compare April 16, 2026 10:49
@SirPyTech SirPyTech marked this pull request as ready for review April 16, 2026 11:04
Copy link
Copy Markdown

@quirino95 quirino95 left a comment

Choose a reason for hiding this comment

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

Code review: LGTM.

Functional review: find a non-blocking issue. I've already approved but if you could, I'd like you to improve this thing:
When selecting a mailing list, I need to save before button "Prune lists" appears.
As consequence, if an user click on "SEND" button directly, won't never see "Prune lists".
Would it be possible to make the button appear when you choose the mailing list instead of waiting for the save?

Blacklisted contacts can be removed while creating a new mailing:

1. Create a new mailing
2. Select a miling list that contains blacklisted contacts: a button "Prune lists..." appears
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Chore (non blocking): small typo here

Suggested change
2. Select a miling list that contains blacklisted contacts: a button "Prune lists..." appears
2. Select a mailing list that contains blacklisted contacts: a button "Prune lists..." appears

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

Labels

mod:mass_mailing_list_prune_blacklisted Module mass_mailing_list_prune_blacklisted series:16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants