Skip to content
This repository has been archived by the owner on Sep 16, 2021. It is now read-only.

[WIP] Moderation queue #61

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

[WIP] Moderation queue #61

wants to merge 8 commits into from

Conversation

rossjones
Copy link
Contributor

Provides a moderation queue to allow for un-spamming and deleting of both comments and issues. It is expected that this will be populated by the akismet integration.

Partially fixes datagovuk/ckanext-dgu#368

Provides a skeleton of a UI at /issues/moderation which allows
the admin to delete spam, or mark it as ham.  When marking items as
ham they need to feed back to akismet to improve the spam checking
service.

Required new action functions to execute commonly required
functions, like getting a comment by ID, deleting comments (although
there was functionality for deleting issues already).

TODO:
 - UI cleanup (pagination?)
 - Controller methods for issue delete/reset
 - Checking of auth tests.
 - Tests
Where issues are being moderated, it is now possible
to clear the report or delete the issue depending on whether
we want to reset the issue or remove it (because it is spam).
The action layer was inconsistent in the formatting of the result,
with issue_comment_search returning a naked list of objects whilst
issue_search returned a dict with a count and a results key.  So
this commit fixes issue_comment_search.

Also adds basic pagination to the moderation controller, although
it is not yet rendered
Changes the issue_comment_search to accept offset and limit and to also
wrap the results with a count.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Integrate Moderation UI for Issues
1 participant