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

Allow disabling moderator email notifications #4543

Closed
jywarren opened this issue Jan 6, 2019 · 4 comments
Closed

Allow disabling moderator email notifications #4543

jywarren opened this issue Jan 6, 2019 · 4 comments
Labels
break-me-up break up for cleaner code separation, discrete tests, and, easier and iterative collaboration enhancement explains that the issue is to improve upon one of our existing features help wanted requires help by anyone willing to contribute
Milestone

Comments

@jywarren
Copy link
Member

jywarren commented Jan 6, 2019

Moderators get a lot of email. We should let them continue to moderate on the site, but disable the emails at https://publiclab.org/settings

In a separate follow-up, we could add a note to to moderators emails linking to that setting. We could offer an alternative method - an RSS feed, or a chat channel to review on, rather than by email.

@jywarren jywarren added enhancement explains that the issue is to improve upon one of our existing features help wanted requires help by anyone willing to contribute break-me-up break up for cleaner code separation, discrete tests, and, easier and iterative collaboration labels Jan 6, 2019
@jywarren jywarren added this to the Spam! milestone Jan 6, 2019
@Anjalizi
Copy link
Contributor

hi @jywarren. can you help me get started on this issue if this is still open?

@jywarren
Copy link
Member Author

Thank you!!!!

Let's add a toggle when sending moderator emails, to look for user.has_tag('block-node-moderation-emails') -- then we can add a UI to add that tag!

The moderators to notify are identified in these sections here:

https://github.com/publiclab/plots2/blob/master/app/mailers/admin_mailer.rb#L6-L30

Would you be interested in trying to add some logic to check for the blocking tag, in the appropriate sections of code there?

@Anjalizi
Copy link
Contributor

I'll start working on this.

@jywarren
Copy link
Member Author

jywarren commented Jan 7, 2020

Merging into #6282

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
break-me-up break up for cleaner code separation, discrete tests, and, easier and iterative collaboration enhancement explains that the issue is to improve upon one of our existing features help wanted requires help by anyone willing to contribute
Projects
None yet
Development

No branches or pull requests

2 participants