Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ChatGPT Auto-answer should be in an embed #917

Closed
Zabuzard opened this issue Oct 22, 2023 · 2 comments · Fixed by #1046
Closed

ChatGPT Auto-answer should be in an embed #917

Zabuzard opened this issue Oct 22, 2023 · 2 comments · Fixed by #1046
Assignees
Labels
enhancement New feature or request priority: normal

Comments

@Zabuzard
Copy link
Member

Its necessary for better UX that ChatGPT answers are posted in embeds with a proper author and icon that clearly marks the content as AI answer.

Right now, its too confusing for users since it looks like a regular message.

The disadvantage is that code snippets are typically harder to read in embeds. We will try it out and see how bad it is. Its likely that we can live with it.

Also, when posting embeds, the text limit is boosted from 2k to 4k characters (? check embed max size). So that would reduce the amount of messages that need to be posted, which is good.

@Zabuzard Zabuzard added enhancement New feature or request priority: normal labels Oct 22, 2023
@Zabuzard
Copy link
Member Author

Would also fix #928 if done properly

@SquidXTV SquidXTV assigned SquidXTV and unassigned SquidXTV Mar 4, 2024
@SquidXTV
Copy link
Member

SquidXTV commented Mar 6, 2024

going to wait for other PRs to get merged first

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority: normal
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants