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

Move addon configuration out of source control/lua files #60

Closed BadgerCode closed 1 year ago

BadgerCode commented 3 years ago

The configuration for this addon currently lives within lua/damagelogs/config/config.lua.

This causes 2 problems:

  1. When updating their copy of the damage logs, server owners can accidentally overwrite their configuration changes with the latest copy of lua/damagelogs/config/config.lua
  2. Server owners using the workshop version are unable to configure the addon


The configuration for this addon should:

wgaylord commented 1 year ago

I may begin to thinker with this in my free time. Just to get some ideas flowing.

Would this be just an extra config tab in the normal manager that is displayed for admins? (Or what ever group you want similar to the AddUser stuff in the current config. Like another bool that would say if you can see the config tab or not)

Or maybe should it be a button (visible only to admins / group allowed to see it) that opens up its own menu with tabs? (Break it down into different groups like: permissions, autoslay , ban, and other options (for the ones that don't really group with the others)

For now I think getting the config out of the lua and into a JSON file in the data folder is a good first step which I plan on working on.

BadgerCode commented 1 year ago

This was done in v3.6.0

No in-game interface to edit configuration, but that's a separate feature