search

CactusDev / CactusBot

An open source, community-written service-agnostic chat bot
MIT License
30 stars 6 forks source link

Command permission changing #249

Open Innectic opened 7 years ago

Innectic commented 7 years ago 
!command add test Testing!
!command permission +test

Syntax isn't final

2Cubed commented 7 years ago

What if we instead used rename?

!command rename stuff things
!command rename test +test

You wouldn't have to specify the + for the old name, but you could optionally supply it for the new one to change permissions.

Alkali-Metal commented 7 years ago

Personally I feel as though this is would be better as a separate Magic:tm: command for easier integration with other services and also being able to do it with other magic commands. (See 175 for example)

Innectic commented 7 years ago

@2Cubed That's a much better way

Innectic commented 7 years ago

@Alkali-Metal This should stay part of the command command because it changes something about commands. Changing built-in command permissions can still happen using this command, it just needs to be implemented.

Alkali-Metal commented 7 years ago

@Innectic it just feels clunky in my opinion to have a !command command changing built-ins. and with using the !config command it could make it easier to understand that it changes both to people who are new to CactusBot