mumbledj icon indicating copy to clipboard operation
mumbledj copied to clipboard

Add admins to bot though commands

Open MrKrucible opened this issue 10 years ago • 8 comments

Sometimes I'd like someone to have !forceskip permissions, but I'd have to go back into my box to do that. So for convenience it'd be nice to see !addadmin ___ or something along those lines.

MrKrucible avatar Sep 29 '15 20:09 MrKrucible

That would be nice. This would probably require admins to be stored in a separate file, but it is certainly doable. I will keep this in mind.

matthieugrieger avatar Sep 30 '15 01:09 matthieugrieger

This wouldn't be so hard to implement with some sed magic. Would you be opposed to changing the format in the config file from something like this:

Admins = "Bob"
Admins = "Alice"

To this:

Admins = {
  "Bob",
  "Alice"
}

Regardless of the answer, I reckon I'll try poking around with this over the next few days.

Gamah avatar Dec 25 '15 03:12 Gamah

That wouldn't work with gcfg would it?

nkhoit avatar Dec 25 '15 04:12 nkhoit

My only opposition to that format is that I'm not sure it would work with the gcfg format. I would have put it in the format you suggested but it wasn't supported by the gcfg format at the time.

On Dec 24, 2015, 7:40 PM, at 7:40 PM, Ron McMahon [email protected] wrote:

This wouldn't be so hard to implement with some sed magic. Would you be opposed to changing the format in the config file from something like this: Admins = "Bob" Admins = "Alice"

To this: Admins = { "Bob", "Alice" }

Regardless of the answer, I reckon I'll try poking around with this over the next few days.


Reply to this email directly or view it on GitHub: https://github.com/matthieugrieger/mumbledj/issues/88#issuecomment-167188785

matthieugrieger avatar Dec 25 '15 04:12 matthieugrieger

Didn't see @nkhoit's message. What he said!

matthieugrieger avatar Dec 25 '15 04:12 matthieugrieger

Nothing I couldn't write some sed magic for, I'm up for the challenge. So long as you're willing to vet and test the changes, I'm willing to pound out a solution that works.

Gamah avatar Dec 25 '15 07:12 Gamah

Go ahead! Just know that it may be removed once I finish the rewrite as the way that configuration is handled will almost certainly change.

matthieugrieger avatar Dec 25 '15 07:12 matthieugrieger

That's fine, just happy to contribute! :)

I'll probably pound this out over the next couple days.

Gamah avatar Dec 25 '15 07:12 Gamah