CodingTrain / Discord-Bot-Examples

Bot Examples for Fall 2023
20 stars 7 forks source link

a joke slash command #21

Closed kfahn22 closed 1 year ago

kfahn22 commented 1 year ago

Oops, I didn't realize my auto-formatter apparently is set to use "" instead of ''

dipamsen commented 1 year ago

This is a great demo for quiz-like interactions which wait for the user response. (Though, the repo is being reworked for supplementing the videos)

supercrafter100 commented 1 year ago

Have you tested this code? Some of the things in there don't look like they're going to do what you intended them to do 😅

kfahn22 commented 1 year ago

I have tested it in my server and it seems to be doing what I intended. The one issue I am having is that the bot.js file asks about feelings. What are you concerned about?

supercrafter100 commented 1 year ago

I have tested it in my server and it seems to be doing what I intended. The one issue I am having is that the bot.js file asks about feelings. What are you concerned about?

That's correct! In the current version of the repository the bot.js contains some additional code Dan tinkered with which did that when you mentioned the bot.

shiffman commented 1 year ago

Hi @kfahn22 this is great! I was about to merge #20. The idea here is that this repo would have the code to match the videos precisely. Other bots or examples would come in future videos and then could be added to the repo. Also, we can create a section in the README to link to other examples or "passenger showcase" bots (though these would ultimately be on the website as well?).

How about linking to your own repo from the README here for now?

kfahn22 commented 1 year ago

I really just wanted to share the code. I realized that you wanted a clean repo with just basics. I am not sure it makes sense to put a link in the README? Should I close pull request and send you a link instead just in case you want to use i?

dipamsen commented 1 year ago

I think this (closed) pr with Supercrafter's comments can be used as a reference later in case Dan wants to refer to it?