Skip to content

Index bans#67

Open
qimiko wants to merge 6 commits intomainfrom
index-ban
Open

Index bans#67
qimiko wants to merge 6 commits intomainfrom
index-ban

Conversation

@qimiko
Copy link
Copy Markdown
Member

@qimiko qimiko commented Apr 18, 2026

index bans. i didn't really test this besides making sure it compiled :p

a ban blocks developers from

  • creating mod versions
  • uploading mods
  • adding devs to a mod
  • being added as a dev to a mod

it does not do anything else (like unlisting/deleting all existing mods).

ban status is not visible to anyone besides index admins, unless they attempt to add that developer as a developer to a mod, which would be a very silly idea. ban reason is made public when a banned user tries to perform a blocked action. everything else is only visible to index admins

@RayDeeUx
Copy link
Copy Markdown

"SiX sEvEn" yes we all get it please just find a new joke to beat to death

@qimiko

This comment was marked as off-topic.

@TheSillyDoggo

This comment was marked as off-topic.

@Fleeym
Copy link
Copy Markdown
Collaborator

Fleeym commented Apr 21, 2026

I think it would be useful if bans had a revoked_at timestamp, that way we have a history of bans for a user. Also could be used for temporary bans that get revoked automatically.

@qimiko
Copy link
Copy Markdown
Member Author

qimiko commented Apr 21, 2026

can add, although does that mean that bans shouldn't be using dev id as the primary key? (so that one dev can have multiple bans). and then i can have /bans return a list of bans instead of one ban and probably also redo the delete endpoint to support that

@Fleeym
Copy link
Copy Markdown
Collaborator

Fleeym commented Apr 21, 2026

Yeah, that's gonna need a bit of refactoring. Delete endpoint should just set revoked_at = now, basically never delete a ban row.

@Fleeym
Copy link
Copy Markdown
Collaborator

Fleeym commented Apr 22, 2026

Another thing - if I manage to merge submission threads bans should also lock the developer from commenting on threads.

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.

4 participants