DerpleMQ2 / rgmercs

Lua version of RGMercs Script
10 stars 20 forks source link

fix: import not honoring spells removed from sets. #60

Closed grimmier378 closed 1 month ago

grimmier378 commented 2 months ago

if you comment out sections and try to import the config as a custom one, the commented out sections get ignored.

Added a flag to the config FullConfig = true/false if the flag is present and true then replace the default config with the custom one. otherwise use the mergeTables function.

example: local _ClassConfig = { _version = "1.0 Beta", _author = "Derple", ['FullConfig'] = true,

grimmier378 commented 2 months ago

I think the first commit is the best option. and people can just copy the config and make changes. either by hand or with configedit since it can edit the file as well.

the FullConfig flag option would be the best and easiest i think. the flag could just be set to false on the base configs. but the user will have it already so they just have to flip it to import their whole config file.

AlgarDude commented 1 month ago

Fwiw, I am also a proponent of full replacement, I can't imagine anyone not starting with the included configs as a reference point and they can be instructed to do so.