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

[FEAT] Add check to verify message is not empty before querying OpenAI #5

Closed
verovaleros opened this issue Feb 19, 2024 · 0 comments · Fixed by #7
Closed

[FEAT] Add check to verify message is not empty before querying OpenAI #5

verovaleros opened this issue Feb 19, 2024 · 0 comments · Fixed by #7
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@verovaleros
Copy link
Member

Is your feature request related to a problem? Please describe.
Some messages from the DB can be empty, and it could break the LLM and also increase the cost of operations.

Describe the solution you'd like
Add a simple validation to check the message is not empty before processing it

Describe alternatives you've considered
We don't want to modify the DB, so this is the best option I could think of.

Additional context
No

@verovaleros verovaleros added the enhancement New feature or request label Feb 19, 2024
@verovaleros verovaleros added this to the v1.0.0 milestone Feb 19, 2024
@verovaleros verovaleros self-assigned this Feb 23, 2024
@verovaleros verovaleros linked a pull request Feb 23, 2024 that will close this issue
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant