AI response should be in embed

Together-Java / TJ-Bot

TJ-Bot is a Discord Bot used on the Together Java server. It is maintained by the community, anyone can contribute.

https://togetherjava.org

GNU General Public License v3.0

100 stars 83 forks source link

AI response should be in embed #1046

Closed ankitsmt211 closed 4 months ago

ankitsmt211 commented 4 months ago

helper method to generate embed
remove AI parser class that partitioned responses & related tests
refactor dismiss button on generated responses
update docs
resolves #917 and possibly #935 (since now response embed contains title)
resolves #928

Here's how new AI generated responses look like:

gpt-response-embed

ankitsmt211 commented 4 months ago

Btw you want to change the user icon to chatgpt icon? Are we even allowed to?

that would be really nice if we could do that, i can't think of a way to do that. Prolly would need store random asset somewhere and get a url perhaps that we can pass for author.

SquidXTV commented 4 months ago

Btw you want to change the user icon to chatgpt icon? Are we even allowed to?

that would be really nice if we could do that, i can't think of a way to do that. Prolly would need store random asset somewhere and get a url perhaps that we can pass for author.

https://en.m.wikipedia.org/wiki/File:ChatGPT_logo.svg https://chat.openai.com/apple-touch-icon.png https://chat.openai.com/favicon-32x32.png

ankitsmt211 commented 4 months ago

I will give these a try, also i linked a new issue #928 since it will be resolved as well so we can touch 3 of the issues with this merge.

update: tried using urls as string but embeds were blank without an avatar, can hopefully be resolved in later PR if it's somehow possible.