Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature] Don't send notifications to suspended users #918

Open
LavishSphere opened this issue Nov 26, 2023 · 2 comments · May be fixed by #985
Open

[Feature] Don't send notifications to suspended users #918

LavishSphere opened this issue Nov 26, 2023 · 2 comments · May be fixed by #985
Labels
Feature Adding a new feature, or substantial improvements on existing functionality Good First Issue Easy Issues suitable for newcomers Medium Priority Needs attention, not urgent

Comments

@LavishSphere
Copy link

Feature Description

It is best not to waste e-mails on users already susepnded when notifying all users on ctrlpanel. Lol, I didn't even know it did that until I checked my verification e-mail mailbox and found a reply with a hate kys message.

Solution Idea

Don't send notifications when notifying all users to suspended users.

Additional Info

No response

@LavishSphere LavishSphere added the Feature Adding a new feature, or substantial improvements on existing functionality label Nov 26, 2023
@OddDevelopment
Copy link

Agreed. Also an unsubscribe link would be nice for people to opt out of emails if for example someone signed them up without there consent.

@sybetjeh sybetjeh added the Low Priority Can wait, not urgent. Handle when possible label Apr 3, 2024
@S0ly S0ly added Good First Issue Easy Issues suitable for newcomers Medium Priority Needs attention, not urgent and removed Low Priority Can wait, not urgent. Handle when possible labels May 25, 2024
@S0ly
Copy link
Member

S0ly commented May 27, 2024

this would actually be a medium priority since its required to have an unsubscribe and email management
but I think I will do a PR for all these email stuff

@AGuyNamedJens AGuyNamedJens linked a pull request Jul 27, 2024 that will close this issue
17 tasks
@AGuyNamedJens AGuyNamedJens linked a pull request Jul 27, 2024 that will close this issue
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Adding a new feature, or substantial improvements on existing functionality Good First Issue Easy Issues suitable for newcomers Medium Priority Needs attention, not urgent
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants