TheXYZNetwork / xAdmin

A free and basic administration system for Garry's Mod.
The Unlicense
28 stars 6 forks source link

Darkrp commands not documented #20

Closed robertdrakedennis closed 4 years ago

robertdrakedennis commented 5 years ago

From what I can see https://github.com/OwjoTheGreat/xadmin/wiki/Commands does not document

https://github.com/OwjoTheGreat/xadmin/blob/master/lua/xadmin/commands/darkrp.lua

This would just generally be useful to add. Also IMO darkrp should be an external module and not added by default.

0wain commented 5 years ago

Oh, so it is. I must have completely forgotten to document them when I accepted the merge request.

I do agree with you for the most part about not having xAdmin come with DarkRP commands by stock. But then what would be the best method for providing them? Considering it's just the 1 file it would seem a bit overkill to give it it's whole on repo, don't you think?

robertdrakedennis commented 5 years ago

With it being a separate module, you can add more darkrp specific commands, potentially adding things that fadmin has aswell. This will allow people to get their /jail command they wanted and allow you to still keep xadmin what it is, lightweight and functional.

0wain commented 4 years ago

Sorry, been a little bust this month. I'm going to go ahead and make a xAdminDarkRP extension based on your suggestion, it does seem like the best idea. Allow for more tailored commands. I appreciate the feedback.