Skip to content
This repository was archived by the owner on Sep 3, 2025. It is now read-only.

fix(models): updating to Pydantic v2 ValidationData#5974

Merged
whitdog47 merged 1 commit into
mainfrom
fix/validators
May 15, 2025
Merged

fix(models): updating to Pydantic v2 ValidationData#5974
whitdog47 merged 1 commit into
mainfrom
fix/validators

Conversation

@whitdog47

@whitdog47 whitdog47 commented May 15, 2025

Copy link
Copy Markdown
Contributor

This PR updates the code to support Pydantic v2 by switching to the new ValidationInfo object for field validators. Key changes include updated import statements and modifications to the validator method signatures in both document and conference models.

@whitdog47 whitdog47 requested review from Copilot and mvilanova May 15, 2025 18:50

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the code to support Pydantic v2 by switching to the new ValidationInfo object for field validators. Key changes include updated import statements and modifications to the validator method signatures in both document and conference models.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/dispatch/document/models.py Updated imports and validator signature in DocumentRead
src/dispatch/conference/models.py Updated imports and validator signature in ConferenceRead

@whitdog47 whitdog47 changed the title updating to Pydantic v2 validationData fix(models): updating to Pydantic v2 ValidationData May 15, 2025
@whitdog47 whitdog47 self-assigned this May 15, 2025
@mvilanova mvilanova added the bug Something isn't working label May 15, 2025
@whitdog47 whitdog47 merged commit 12d1871 into main May 15, 2025
9 of 12 checks passed
@whitdog47 whitdog47 deleted the fix/validators branch May 15, 2025 19:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants