Multiple flags per reviewmoderation
Bug #675504 reported by
Michael Nelson
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ratings and Reviews server |
Fix Released
|
Undecided
|
Michael Nelson |
Bug Description
In addition to Review.hide, we need a status to the ModerationRequest (ie. New, Confirmed, Rejected etc.), as well as the ability to flag a review multiple times, recording the details.
Also, it might be worth considering one of the generic django-moderation apps that are around (ie. https:/
description: | updated |
Changed in rnr-server: | |
status: | Incomplete → In Progress |
assignee: | nobody → Michael Nelson (michael.nelson) |
summary: |
- Is Review.hide redundant or ModerationRequest itself? + Multiple flags per reviewmoderation |
description: | updated |
Changed in rnr-server: | |
status: | In Progress → Fix Committed |
Changed in rnr-server: | |
milestone: | none → 11.01 |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
I've just spent some time investigating both django-moderation and django-gatekeeper.
Both have lots of useful functionality, but also miss one-or-two features we'd need.
django-moderation /github. com/dominno/ django- moderation
===============
url: https:/
Overview: The "normal" use-case with this app is that new objects require moderation by default. Although this behaviour can be overridden, it seems that there is currently no field on the ModeratedObject model to store who flagged/requested the moderation. Otherwise this app has most features we'd need (and some that we'd not need - such as re-moderating when an object is edited and displaying the changes).
django-gatekeeper /github. com/sunlightlab s/django- gatekeeper
===============
url: https:/
Overview: This app *does* support flagging - great, seems to match our requirements and is very configurable, but doesn't support a reason for flagging in its model.
So we could either branch and contribute to one of these, or start with the simple functionality that we already have (perhaps moving it into a separate django app though) and learn from the work of these other projects. If in the end we can replace our app with one of those - great.