start-again / spookyBot

🎃 A spooky Discord bot
MIT License
9 stars 12 forks source link

Make the prefix configurable #26

Open tmttn opened 3 years ago

tmttn commented 3 years ago

Feature Request

Is your feature request related to a problem? Please describe. Making the prefix configurable allows the server admin to personalize it.

Describe the solution you'd like Store the prefix in the database per server. If no prefix is configured, use the default one. Validate the prefix for the format prefixName!.

Describe alternatives you've considered None 🤔

Teachability, Documentation, Adoption, Migration Strategy

config: {
    command: 'setprefix',
    name: 'Set bot prefix',
    description: 'Change the default bot prefix',
    usage: `${prefix}setprefix <prefix>`,
    displayHelp: true,
    permissionNeeded: 'ADMINISTRATOR',
  },
tmttn commented 3 years ago

I would like to work on this ofc. ;-)

LucasCtrl commented 3 years ago

At the moment, I don't think this feature is necessary. However, I'll leave it open if the necessity arises.

tmttn commented 3 years ago

I was trying out an implementation for this, but it became annoying very quickly, especially in the status / help messages... 😭

ghost commented 3 years ago

I don't think this is necessary, since there isn't any other bot with this prefix. Anyway, it's a good idea to implementate it. PRs welcome, and Lucas could do something too

ghost commented 3 years ago

I would do this with a JSON db, just like this: {"serverId": "prefix" | "default prefix"}

tmttn commented 3 years ago

That's what I was doing, and that part is fine. It's mostly the status message that was giving me a headache, because that one is not server specific. It's specific to the bot user in general.