Skip to content

Spam: make admin filters easier to understand #8688

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

Merged
merged 2 commits into from
Nov 18, 2021

Conversation

humitos
Copy link
Member

@humitos humitos commented Nov 17, 2021

Use a gte and lt applied to the spam filters so we show "only projects
with score between N-M instead of >N"

Use a `gte` and `lt` applied to the spam filters so we show "_only_ projects
with score between N-M instead of >N"
@humitos humitos requested review from astrojuanlu and a team November 17, 2021 10:01
@humitos humitos force-pushed the humitos/spam-admin-filters branch from a83d29b to e168cd7 Compare November 17, 2021 10:02
Copy link
Contributor

@agjohnson agjohnson left a comment

Choose a reason for hiding this comment

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

Makes sense, though it wasn't clear if the first level, "not enough score", was meant to describe legitimate projects or not.

Copy link
Member

@ericholscher ericholscher left a comment

Choose a reason for hiding this comment

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

This would be useful, I wanted this earlier today 👍

@humitos humitos merged commit 2cab501 into master Nov 18, 2021
@humitos humitos deleted the humitos/spam-admin-filters branch November 18, 2021 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants