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

AI Assistant: Handle apollo errors #2474

Open
josephjclark opened this issue Sep 9, 2024 · 3 comments
Open

AI Assistant: Handle apollo errors #2474

josephjclark opened this issue Sep 9, 2024 · 3 comments

Comments

@josephjclark
Copy link
Contributor

The AI assistant needs to show good feedback to users when apollo is offline.

Right now, if the deployment isn't set up (ie, no env vars), there's a nice error page saying "hey, apollo isn't set up, call your sysadmin"

If apollo is not found (if the server does not return 200), the chatbox is greyed out with no feedback.

Error Cases

Here are some errors I can think of:

  • Errors form the apollo server for new chat messages (wrong api key, a bug on apollo, anthropic offline)
  • apollo server isn't responding (should we maintain a heartbeat rather than just asking when the server starts?
  • apollo messages timing out

Expected behaviour

Even if apollo is unavailable, the user should be able to see and scroll the chat history

I think basically we need to:
a) anticipate that we can't send a message and disable the chat box with an error
b) handle requests which return errors (do we save the unsent message in history? When do we re-send?)

@github-project-automation github-project-automation bot moved this to New Issues in v2 Sep 9, 2024
@christad92
Copy link

When connection to Apollo fails, we should show the users a message saying: Oops! Couldn't connect to the AI Assistant, please contact your instance administrator.

We should also block the user of the feature until the server sends a message back

@josephjclark
Copy link
Contributor Author

@christad92 We should still show chat history though! Don't block the user from the assistant completely - just stop them sending messages

@josephjclark
Copy link
Contributor Author

Maybe rleated to #2704

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Backlog
Development

No branches or pull requests

2 participants