search

carsonwilliamsdev / bawt

My Discord Bot
MIT License
5 stars 0 forks source link

Refactor the commands #13

Closed boveus closed 5 years ago

boveus commented 5 years ago

This isn't ready to merge in yet.

I was trying to refactor the commands out of index.js to make things easier to maintain. It looks the commands that use promises are failing to return anything. I did some testing and it looks like this refactor is causing the commands to return before the promise resolves, so they end up returning undefined.

You have any ideas for why this is happening?

boveus commented 5 years ago

Okay this should be fixed now. I just had to move the message.send to be in the commands themselves. I think we can clean this up a bit more to make it a little easier to add commands, but this should be a step in the right direction.