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

Plugin support for sophisticated spam handling #6692

Open
wants to merge 19 commits into
base: develop
Choose a base branch
from

Conversation

kontrollanten
Copy link
Contributor

Description

Add support for plugins to filter action menus on:

  • Admin user list, bulk and single
  • Admin comment list, bulk and single
  • Admin abuse list, single
  • Admin user moderation (available in multiple views)

Add support for doAction plugin API. Supported actions:

  • Increment/decrement loader
  • Admin user list, load data
  • Admin video comment list, load data
  • Admin user list, load data
  • Admin abuse list, load data
  • Video watch comment list, load data

Proof of concept with a plugin available at https://framagit.org/kontrollanten/official-plugins/-/merge_requests/new/diffs?merge_request%5Bsource_branch%5D=feat-akismet-mark-as-spam

Related issues

closes #6264

Has this been tested?

  • 🙅 no, because this PR does not update server code

Screenshots

image
image
image
image
image
image
image

Copy link
Owner

@Chocobozzz Chocobozzz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

client/src/app/app.component.ts Show resolved Hide resolved
client/src/app/app.component.ts Show resolved Hide resolved
client/src/app/core/plugins/plugin.service.ts Outdated Show resolved Hide resolved
client/src/app/core/plugins/plugin.service.ts Outdated Show resolved Hide resolved
packages/models/src/plugins/client/client-hook.model.ts Outdated Show resolved Hide resolved
@kontrollanten
Copy link
Contributor Author

@Chocobozzz All comments fixed. The failing tests seem to be the same as in develop branch.

@kontrollanten
Copy link
Contributor Author

I did a last (?) update with changing action admin-user-list:load-data to admin-users-list:load-data and also updated the docs. I tried to update the dedicated API docs but got this error when I referenced to the new file client-action.model.ts:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Plugin support for adding action links to users and comments
2 participants