zmoin / slapbot Goto Github PK

It is possible to get the bot to leave a react on messages.
https://discord.js.org/#/docs/main/stable/class/Message?scrollTo=react
Get the bot to react to command messages with a thumbs up or smiley face or something.

To further expand on the idea, I've included a package called chance.js you could use this to pick a random emoji.
https://chancejs.com/helpers/pickone.html

We are sorely lacking in commands. Maybe we should have a second?

Currently the bot will accept any string as input for it's slap subject.
Instead it should accept a user @ to get a users data then @ that passed in user in the response.
It appears you can get user mentions through a member on the message object passed into the event callback.
https://discord.js.org/#/docs/main/stable/class/Message?scrollTo=mentions

The project repo currently doesn't have a licence for its code to signify what people may do with the code contents.
Below appears to be a guide on adding a licence
https://help.github.com/en/articles/adding-a-license-to-a-repository

Can find out more by googling software/code licencing. And open source licencing.

node_modules should be ignored.

Using .slap with two mentions sometimes causes it to use one for both slappee and slapper.
Given that the .first() method uses an iterator behind the scenes, might be best to use something else.

The data structure being used has an .array() method, so it might be best to use that instead to get the array of mentions, and work from there.

I tested it locally, the array method is really messy, and not a viable solution.

Edit: Haven't tested it too thoroughly, but what you've done seems to be a good solution. So this can probably be marked as closed now.

The instructions are hard to read. Put them on new lines or bullet point them

In addition to the existing blacklist and whitelist, add a third list of key pairs so we can denote permission levels.
example being having Zarrin set as admin being able to use special secret new commands, whereas everyone else is considered a default user who can't use special secret new commands.

As an exploration of use of the whitelist/blacklist. Add more ids to them so that we can better moderate command use.