Zachdehooge / NadoBot

A Discord.py bot that pulls nadocast data
3 stars 2 forks source link

Help command #6

Open JacksonDorschner opened 1 month ago

JacksonDorschner commented 1 month ago

we need to make a nice looking help command, most likely using embeds.

Arch881010 commented 1 month ago

image There is already a built in help command, we either need to decide to leave it that way (as we can remove it and create our own help command) or our way. Need feedback. (The current $help command can be changed, where the command categories will have their own categories, or the info needed in the help command.)

Arch881010 commented 1 month ago

image This is an example of what we can do.

JacksonDorschner commented 1 month ago

although more work, what I think would look better is with an embed. I think most things should be using embeds because they are more user friendly with the "new" embed system, and look way better. I think you guys get the final call though.

Arch881010 commented 1 month ago

I think the embed system would be better from a GUI standpoint. I just know we have to write over the help command (as per discord.py docs). We can use a new class to overwrite it (checkout this stackoverflow example) or by first removing the command (bot.remove_command("help")) then writing a new one (like any other command) (as per this stackoverflow answer).

JacksonDorschner commented 1 month ago

you can see my incomplete branch that has started figuring out how to implement the embed

Arch881010 commented 1 month ago

you can see my incomplete branch that has started figuring out how to implement the embed

I do not see this branch.

JacksonDorschner commented 1 month ago

I have been pushing to my own fork lol, I will open another branch on this repo in a few hours.

Arch881010 commented 1 month ago

I have been pushing to my own fork lol, I will open another branch on this repo in a few hours.

After reading said fork, I don't think it's going in the direction I expected. Given sometimes there is more than 1 image.

JacksonDorschner commented 1 month ago

I was thinking we could have 2 images in one embed, and have a text field with a brief explanation of both images.