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

fix: enforce json schema for prompts and conversations #253

Merged
merged 3 commits into from
Mar 1, 2024

Conversation

Maxim-Gadalov
Copy link
Contributor

@astsiapanay
Copy link
Collaborator

May any fields be optional in the JSON schema? If so we shouldn't throw an exception and mark these fields optional.

@astsiapanay
Copy link
Collaborator

Should we ignore unknown fields?

astsiapanay
astsiapanay previously approved these changes Mar 1, 2024
@Maxim-Gadalov
Copy link
Contributor Author

Maxim-Gadalov commented Mar 1, 2024

/deploy-review

Environment URL: https://chat-ai-dial-core-pr-253.nightly-test.deltixhub.io
E2E tests status: failed

@Maxim-Gadalov
Copy link
Contributor Author

Maxim-Gadalov commented Mar 1, 2024

/deploy-review

Environment URL: https://chat-ai-dial-core-pr-253.nightly-test.deltixhub.io
E2E tests status: failed

@Maxim-Gadalov Maxim-Gadalov merged commit 257c2a3 into development Mar 1, 2024
7 checks passed
@astsiapanay astsiapanay deleted the issue-213 branch August 9, 2024 15:54
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.

2 participants