Resonite-Community-Projects / resonite-pterodactyl

5 stars 4 forks source link

Minimize the uneeded configuration properties #9

Closed ProbablePrime closed 7 months ago

ProbablePrime commented 7 months ago

I noticed that the final output of Pterodactyl includes a bunch of unneeded properties.

Anything in config.json whose end result is null, "" or [], or even {}, should be omitted from the final configuration, this makes bug hunting easier and keeps the cognitive load when reading instances low.

Could this be re-worked to exclude the unneeded parameters?

ProbablePrime commented 7 months ago

In particular, the adult session related matters should also be omitted.

ProbablePrime commented 7 months ago

https://github.com/Resonite-Community-Projects/resonite-pterodactyl/pull/10

GrayBoltWolf commented 7 months ago

Removed some unreferenced parameters from the default template.

Note the remaining items may be null in the template, however are referenced in the egg configuration.

GrayBoltWolf commented 7 months ago

Updated