Add admins to bot though commands

[MrKrucible]

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.

[matthieugrieger]

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.

[Gamah]

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.

[nkhoit]

That wouldn't work with gcfg would it?

[matthieugrieger]

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 notifications@github.com 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:
#88 (comment)

[matthieugrieger]

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

[Gamah]

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.

[matthieugrieger]

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.

[Gamah]

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

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