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

[feature]:AI Chatbot support #2727

Open
wants to merge 35 commits into
base: master
Choose a base branch
from

Conversation

hritikchaudhary
Copy link

What's changed?

I've added an ai chatbot support, I'm unable to config the ai endpoints as I wasn't able to translate the ai config docs

closes #2303

Checklist

  • I have read the Contributing Guide
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed.

Add or update API

  • I have added the necessary e2e tests and all cases have passed.

@hritikchaudhary
Copy link
Author

Screen.Recording.2024-09-20.at.1.51.33.AM.1.1.mov

@Yanshuming1
Copy link
Member

Screen.Recording.2024-09-20.at.1.51.33.AM.1.1.mov

Ok, I'll test it later

@Yanshuming1 Yanshuming1 changed the title AI Chatbot support [feature]:AI Chatbot support Sep 20, 2024
@hritikchaudhary
Copy link
Author

Sure, please let me know the feedback I'll accommodate the changes.

@Yanshuming1
Copy link
Member

Sure, please let me know the feedback I'll accommodate the changes.

Take a look at the code, ci fail

@Yanshuming1 Yanshuming1 marked this pull request as ready for review September 25, 2024 13:56
@Yanshuming1
Copy link
Member

Sure, please let me know the feedback I'll accommodate the changes.

Here are a few questions to look at:
1、Fix the ci failure issue
2、The back end returned sse streaming data, the front end did not display it and reported an error
image

@tomsun28 tomsun28 added the good first pull request Good for newcomers label Oct 16, 2024
@tomsun28 tomsun28 added this to the 1.6.2 milestone Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

[Task] Make a page to support sse stream return at the back end
7 participants