Open-Assistant icon indicating copy to clipboard operation
Open-Assistant copied to clipboard

Trollboard: Option to Hide Banned Users

Open blankster opened this issue 2 years ago • 6 comments

The moderator team as well as the "spam" volume is getting bigger and bigger. To collaborate nicely together and to prevent spam/unwanted entries and inefficient overhead in collaboration, I would appreciate it if the already banned users could (optionally) be hidden on the troll board.

This would allow a quick overview of critical users on the trollboard, instead of a view that is polluted by already banned users, and makes early detection of problematic users more difficult because this way all moderators have to go through all users again and again to evaluate whether someone is banned or not.

Early detection is important to avoid wasting the time of constructive community members and moderators.

blankster avatar Feb 20 '23 09:02 blankster

Hi, just to get a clear picture of this request, I created a "megatroll" user, which ends in the Trollboard cause negative votes on his post

image

So if this user gets its Role changed to "banned"

image

We should not longer see him in the Trollboard list

Is that correct?

ArgiesDario avatar Feb 20 '23 15:02 ArgiesDario

I added the necessary backend changes here: https://github.com/LAION-AI/Open-Assistant/pull/1765 You can add enabled=true to the trollboards query to only retrieve enabled users, don't include send the query param to return all users and set it to false to only return disabled users. I think in the front-end it would be best to have a checkbox to toggle the filter option.

andreaskoepf avatar Feb 20 '23 16:02 andreaskoepf

@ArgiesDario I was also thinking about a checkbox or something like that. To have the option to choose this, but not being forced to never see the banned users again. Both views can have their advantages while moderation.

blankster avatar Feb 20 '23 17:02 blankster

I added the necessary backend changes here: #1765 You can add enabled=true to the trollboards query to only retrieve enabled users, don't include send the query param to return all users and set it to false to only return disabled users. I think in the front-end it would be best to have a checkbox to toggle the filter option.

I was trying to make your endpoint work for a while, but it seems to be ignoring the enabled query param, still getting "enabled": false users as response

image

If I hardcode your line 17 in trollboards.py from enabled: Optional[bool] = None to enabled: Optional[bool] = True it works as expected, so it seems to be some related to the get query params. Am I missing something?

ArgiesDario avatar Feb 21 '23 04:02 ArgiesDario

I was trying to make your endpoint work for a while, but it seems to be ignoring the enabled query param, still getting "enabled": false users as response

It seems you are running requests against the frontend-webserver, but the correct backend-url is /api/v1/trollboards/{time_frame} e.g. if you start the local dev setup it would be something like http://localhost:8080/api/v1/trollboards/day?max_count=100&enabled=true ... you need to add the enabled query param here: https://github.com/LAION-AI/Open-Assistant/blob/5982b1b0411a3a261d374d3aea642966123e1f76/website/src/lib/oasst_api_client.ts#L392-L396

andreaskoepf avatar Feb 21 '23 07:02 andreaskoepf

@andreaskoepf thanks!👍, I'm still learning how the project works

It is working now. I added a radio button that allows to show active/enabled users (this option is the default one), and other radio that only shows banned users.

Enabled radio is green, banned is red. Here some screenshots on how it looks:

Dummy User is an active user imagen

Megatroll is a banned user imagen

This is how it looks with white background and spanish translations imagen

I'll be sending the PR soon

ArgiesDario avatar Feb 21 '23 19:02 ArgiesDario

@ArgiesDario I can't thank you enough for this, that's super helpful :)

blankster avatar Feb 23 '23 20:02 blankster