-
-
Notifications
You must be signed in to change notification settings - Fork 893
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
Feature filter keywords 3710 #5263
base: main
Are you sure you want to change the base?
Conversation
What happens if you block "X"? Will it match "Xylophone"? What about "x.com"? Unlike person and community blocks this seems a bit more ambiguous. |
In the moment yes, because the ilike searches for case insensitive substrings. |
You could also consider a minimum length of eg 3 characters for each blocked string. And you need to make sure that the checks are passing (click the details link at the bottom of this page). See .woodpecker.yml in the repo for the commands it runs, so you can check and fix them locally. |
yes this sounds more simple and in the Connect Lenny app they do it like this. I will add the restriction, fix checks add unit tests when I have time at the beginning of the next year. |
…nding functions in diesel
0441a7d
to
3d1e7d5
Compare
I apologize for being slow to review this, ping me or request review when you'd like me to take a look. |
Option to block keyswords so no post containing this keywords in their name, body or url are returned, keywords are stored in seperate table related to person