Okay. It’s still unclear to me why piracy was not picked up by default. It’s a very big comm.
Anyhow, good luck on the merge request! Would be great to let the admin decide what to block instead of the weirdly random selection of comms and words on the list now.
Not block. Just karma-devalue. I’m gonna rename the variable to something like “unwanted reputation sources”. If an admin doesn’t want people farming rep by posting porn, they can put porn on the list. That’s not blocking, it’s just a data filter for admin eyes only.
Okay. It’s still unclear to me why piracy was not picked up by default. It’s a very big comm.
Anyhow, good luck on the merge request! Would be great to let the admin decide what to block instead of the weirdly random selection of comms and words on the list now.
Not block. Just karma-devalue. I’m gonna rename the variable to something like “unwanted reputation sources”. If an admin doesn’t want people farming rep by posting porn, they can put porn on the list. That’s not blocking, it’s just a data filter for admin eyes only.
So there are two purposes for this “bad list”
correct?
The federation filtering is blocking IMO. It affects what comms users of the instance can find (via search).
Anyway maybe in Your PR You should make a separately configurable list for each purpose. I don’t see a reason why it should be the same list.
No, just one purpose, the second one.