这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@jonathanortega2023
Copy link

Pull Request Type

  • 🐛 fix

Relevant Issues

resolves #4606

What is in this change?

Checks if vecDB selection from ENV is valid, logs and toasts when invalid and falls back onto LanceDB as the selection with unsaved changes.

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
  • Docker build succeeds locally

@timothycarambat
Copy link
Member

This "bug" is only possible by someone opening the ENV file directly, typing in an incorrect value, and then opening the app. It is not possible to select an invalid provider from the UI. Additionally, this only patches the UI crash and not the underlying issue since even if the UI is right we should failover to LanceDB on the backend as well - not just show that on the frontend.

The user can then wind up seeing lanceDB in the frontend due to an invalid value, but the backend upserts still fail!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: VecDB page doesn't catch erroneous env selections

2 participants