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

Create spam management for comments #428

Closed
4 tasks
jywarren opened this issue Mar 19, 2016 · 4 comments
Closed
4 tasks

Create spam management for comments #428

jywarren opened this issue Mar 19, 2016 · 4 comments
Labels
enhancement explains that the issue is to improve upon one of our existing features

Comments

@jywarren
Copy link
Member

The problem

Comments spam persists on the PublicLab.org site even when their author has been banned. Individual comments cannot be marked as spam, so they must be deleted.

Proposed fix

  • add comment.status column
  • mark all banned users' comments as status = 0 to unpublish
  • add where(status: 1) to all comment displays
  • add buttons to comments, visible to mods and admins, for spamming and unspamming comments
@jywarren jywarren added the enhancement explains that the issue is to improve upon one of our existing features label Mar 19, 2016
@Fastie
Copy link

Fastie commented Mar 21, 2016

While we are waiting for these enhancements, moderators could remember that after a research note is marked as spam, any comments on that note should be deleted so they don't appear in the list of recent comments..

@wimpykid26
Copy link
Contributor

@jywarren Hi!. This seems like something neccessary and doable. Could you kindly point me towards the neccessary code in order to do this. Also, How do I know that a particular use who is commenting , is banned ?. Thanks in advance. :)

@jywarren
Copy link
Member Author

Users each have a status category which should be 1 -- docs here: https://github.com/publiclab/plots2/blob/master/app/models/drupal_users.rb

Great, please break this up into smaller sub-issues, so that multiple people can work on it at once, and also so that it's easier to assess (and write tests for) each step -- thank you! Moderation functions are especially important to thoroughly test and though I'm traveling at the moment and less able to help day-to-day, I can take some time to advise on good testing as we go. Also perhaps @publiclab/reviewers could contribute their thoughts. Start with unit tests, and good luck!

@jywarren
Copy link
Member Author

Merging into #1785 -- Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement explains that the issue is to improve upon one of our existing features
Projects
None yet
Development

No branches or pull requests

3 participants