Skip to content
This repository has been archived by the owner on Sep 2, 2024. It is now read-only.

Add support for DeathMatch.TF and administration-agnostic bans #187

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Naleksuh
Copy link

@Naleksuh Naleksuh commented Sep 1, 2023

DeathMatch.TF uses a custom, proprietary, administration system, which isn't present in SourceBans. However, adding every single one is probably not reasonable, which is why I added administration-agnostic bans via the sm_ban command-- simply having the system do things the same way the users do.

@sapphonie
Copy link
Contributor

This is not ban system agnostic, and shouldn't be tied to a DMTF cvar. I'd accept it if it was something like lilac_use_console_bans or something

@Naleksuh
Copy link
Author

@sapphonie The way it's set up is to check if it exists, and if it doesn't, use the generic one. That means that it should work fine on any server that isn't DMTF-oriented. Is it not working for you? If so, I can edit it to only use the console ones-- which do work, anyway.

@sapphonie
Copy link
Contributor

@sapphonie The way it's set up is to check if it exists, and if it doesn't, use the generic one. That means that it should work fine on any server that isn't DMTF-oriented. Is it not working for you? If so, I can edit it to only use the console ones-- which do work, anyway.

I was referring to the cvar names, as well as the inclusion of a library that is used by exactly one server network.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants