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/catch validation errors #83

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

Conversation

KyrianC
Copy link
Collaborator

@KyrianC KyrianC commented Nov 23, 2023

Catch validation errors on pydantic models used for providers responses standardization

The ValidationError will be logged instead and the invalid field will be set to None

Allow us to still get a partial response instead of receving an error when provider return unexpected fields

@KyrianC KyrianC requested review from SamyMe and floflokie November 23, 2023 14:57
@KyrianC KyrianC force-pushed the feat/catch-validation-errors branch from 7d1fddf to 86f7834 Compare November 23, 2023 15:18
@KyrianC KyrianC force-pushed the feat/catch-validation-errors branch from 86f7834 to c89e83d Compare November 23, 2023 16:40
@KyrianC
Copy link
Collaborator Author

KyrianC commented Jan 26, 2024

up

@KyrianC
Copy link
Collaborator Author

KyrianC commented Jan 26, 2024

@SamyMe

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.

1 participant