#8020 Easy way to view all posts from a certain user, and flag as spam

v1.4.0
closed
Igor Bondarenko
42cc (432) sf-4 (350) sf-current (14)
General
Dave Brondsema
2016-01-27
2015-11-10
Dave Brondsema
No

The moderation page should have a field to filter by a username - then it will be easier to get a list of all the posts by a user in that forum, and select-all to mark as spam.

Also, when in the regular view of a post and you click "Spam" and the message goes away, we should show a little message box in its place that explains what just happened, and a link to go over to the moderation view and see all posts by the user (so project admins know that is possible and have an easy way to get there). An undo link there would be very nice too.

Related

Tickets: #2914
Tickets: #5028

Discussion

  • Igor Bondarenko - 2015-12-09
    • labels: --> 42cc
    • status: open --> in-progress
    • assigned_to: Igor Bondarenko
     

  • Dave Brondsema

    Dave Brondsema - 2015-12-14
    • labels: 42cc --> 42cc, sf-4, sf-current
     

  • Igor Bondarenko - 2015-12-24
    • status: in-progress --> review
     

  • Igor Bondarenko - 2015-12-24

    Closed #874, #875

    ib/8020a

     

  • Dave Brondsema

    Dave Brondsema - 2016-01-06
    • status: review --> in-progress
    • Reviewer: Dave Brondsema
     

  • Dave Brondsema

    Dave Brondsema - 2016-01-06

    I have removed the 'flags' input on the moderation page as well as dead code related to it. Can you check it over on db/8020?

    This looks good and the only issue I found is that the undo doesn't properly update the number of posts in the thread. This number can be seen in the forums list of threads. In def moderate the undo action just sets the status, but the other actions call methods which do several things including update self.thread.num_replies or call update_stats.

     

    • Igor Bondarenko - 2016-01-07

      "flags" changes look good to me. Will fix posts count

       

  • Igor Bondarenko - 2016-01-21
    • status: in-progress --> review
     

  • Igor Bondarenko - 2016-01-21

    Closed #886. ib/8020b

     

  • Dave Brondsema

    Dave Brondsema - 2016-01-22
    • status: review --> in-progress
     

  • Dave Brondsema

    Dave Brondsema - 2016-01-22

    Why is the -1 removed? Was it kind of like a bug before maybe, and the flush() calls change the behavior or something?

    I got the wrong counts doing this:

    • create new discussion topic, 3 comments in it
    • overview page says there are 3 posts, correct
    • mark the last one as spam, it says there is only 1, incorrect
    • hit "undo", it now says there is 2, still incorrect
     

    • Dave Brondsema

      Dave Brondsema - 2016-01-22

      Also, the tests that had num_replies values changed is a little concerning. Unless the tests themselves were wrong before, those numbers shouldn't have to change.

       

  • Igor Bondarenko - 2016-01-26

    Hm... I saw the exact behavior you described before -1 was removed, that is the reason it was removed. I re-checked just now and this scenario works ok for me (3 initial posts -" spam" changes to 2 - "undo" changes back to 3). Could you re-check? Maybe make sure docker containers are restarted and running updated code? Thanks

     

  • Dave Brondsema

    Dave Brondsema - 2016-01-27
    • status: in-progress --> closed
     

  • Dave Brondsema

    Dave Brondsema - 2016-01-27

    Works for me now, after ensuring everything was updated.

     

  • Dave Brondsema

    Dave Brondsema - 2016-04-11
    • Milestone: unreleased --> v1.4.0
     

Log in to post a comment.