search

winston-yallow / godot-discord-bot

Small bot written for the Godot discord server
MIT License
3 stars 2 forks source link

WIP: Add Vulkan Error #11

Open Xananax opened 4 months ago

Xananax commented 4 months ago

Following on https://discord.com/channels/212250894228652034/1210613462288437288

I'm not too sure about the command name, and neither about what the client accepts in terms of formatting (can it do markdown links? Or do we have to use setUrl?), so this is just a first draft.

winston-yallow commented 2 months ago

Hmm, thinking about this more I'm not so sure anymore if it's good adding such a specific command to the bot. I feel like this would be better suited for the docs (also for maintaining the information), and then maybe a command that links there.

(I won't be blocking this if it's wanted, just a thought that I had)

winston-yallow commented 2 months ago

I'm not too sure about the command name

I would call it vulkan-troubleshooting. But vulkan-error is fine too, don't think it matters a lot as it's a slash command and has a description showing up in the UI.

what the client accepts in terms of formatting (can it do markdown links? Or do we have to use setUrl?

Everything that a normal message can contain should be fine, so markdown links will work. setUrl would make the whole thing clickable, not only the specific text snippet.

Xananax commented 2 months ago

I don't actually know if it's wanted, in my opinion it's not really a great place, but nanachi wanted it. Since we agreed on it on Discord, and it's been a while, I suggest to merge nonetheless, it'd be a bit bothersome to come back a month later and say "actually...". But also not insisting either way.

winston-yallow commented 1 month ago

Alright, then let's merge this. Any objection if I change the command name to something like /troubleshoot vulkan-error? We can then later include other troubleshooting subcommands

Xananax commented 1 month ago

That sounds good to me, but I'm currently too overwhelmed to get back on this atm. If you don't do it, I'll get back to it at some point in the next few weeks.