BadgerCode / tttdamagelogs

Damagelogs and RDM Manager for Trouble in Terrorist Town (a Garry's Mod gamemode)
GNU General Public License v3.0
20 stars 23 forks source link

Add two new configs settings, Check Custom by default and Allow banning fromRDMManager #73

Closed wgaylord closed 1 year ago

wgaylord commented 1 year ago

The first new config settings is because we updated our copy of tttdamagelogs that use to default to having the Custom Reason box checked by default and found we had to edit the lua to make it checked after updating. (It was driving our staff nuts having to remember to check it.)

I have that setting defaulting to false so that it follows what is already set up in the addon currently.

I have also added a new setting to remove the ban option from the Actions menu, since we have had a few people accidentally ban people when they were trying to issue a slay. This is defaulted to true so that the functionality of the addon is not change unless you change the config.

BadgerCode commented 1 year ago

Heyo! Thanks for raising this! I shall take a look at it tomorrow

wgaylord commented 1 year ago

Btw this does not fix the error I mentioned in the other PR that was recently merged about missing ends in oldlogs.lua (didnt notice that initally)

BadgerCode commented 1 year ago

Nice work!

I shall test this in the next-version branch. I have also fixed the issue with the oldlogs.lua file.