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

Notification message as broadcast email. #9323

Closed
wants to merge 1 commit into from

Conversation

per-garden
Copy link

Accidentaly closed #8309

@jvanbaarsen
Copy link
Contributor

@randx For #8309 you said you need to investigate something. Did you already have time for that?

@dzaporozhets
Copy link
Member

@jvanbaarsen nope. So far I see no reason emailing User.all.

@jvanbaarsen
Copy link
Contributor

@randx Ok thanks for your response. Is the fact that a lot of users are emailed in a batch bothering you? Or the fact that also blocked users are being emailed?

@dzaporozhets
Copy link
Member

@jvanbaarsen the fact that we emails all users. I don't know any service that send me email about their maintenance. The concept of broadcast messages does not match with emailing all users for me.

@jvanbaarsen
Copy link
Contributor

@randx Ok thanks. @per-garden I'll be closing this PR for now. If you feel that it should be implemented, please open a feedback item on feedback.gitlab.com so we can discuss it there.

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

Successfully merging this pull request may close these issues.

3 participants