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

NRL-786 new parent class that does not allow extra fields for pydantic models #781

Open
wants to merge 11 commits into
base: develop
Choose a base branch
from

Conversation

eesa456
Copy link
Contributor

@eesa456 eesa456 commented Dec 19, 2024

No description provided.

Copy link

💥 Something went wrong while building the pull request environment.
Check Output Logs

Copy link

💥 Something went wrong while deploying the pull request environment.
Check Output Logs

Copy link

🚀 PR environment successfully deployed.
Commit Hash: 2f6eac9a53dee48e897c4f870629ec3126c2860b
URL: https://nrl786-4ab1a4.api.record-locator.dev.national.nhs.uk/

Copy link

🚀 PR environment successfully deployed.
Commit Hash: 7be77386b2c5019593044165bb31b6bc3d6ee616
URL: https://nrl786-4ab1a4.api.record-locator.dev.national.nhs.uk/

Copy link

🚀 PR environment successfully deployed.
Commit Hash: d50ae24dfeadfbea6351e3331fb7dc0c4d35697d
URL: https://nrl786-4ab1a4.api.record-locator.dev.national.nhs.uk/

Copy link

🚀 PR environment successfully deployed.
Commit Hash: 93025f14b0bfc0ae203f5a754c37e1ebfbeb4ec7
URL: https://nrl786-4ab1a4.api.record-locator.dev.national.nhs.uk/

Copy link

github-actions bot commented Feb 7, 2025

💥 Something went wrong while building the pull request environment.
Check Output Logs

Copy link

github-actions bot commented Feb 7, 2025

💥 Something went wrong while building the pull request environment.
Check Output Logs

Copy link

github-actions bot commented Feb 7, 2025

💥 Something went wrong while building the pull request environment.
Check Output Logs

Copy link

sonarqubecloud bot commented Feb 7, 2025

Copy link

github-actions bot commented Feb 7, 2025

💥 Something went wrong while deploying the pull request environment.
Check Output Logs

Copy link

github-actions bot commented Feb 7, 2025

🚀 PR environment successfully deployed.
Commit Hash: ec405a5fbff8d8def0aae399b4336a3c2ed6b404
URL: https://nrl786-4ab1a4.api.record-locator.dev.national.nhs.uk/



class ParentExtension(BaseModel):
valueCodeableConcept: Annotated[
Copy link
Collaborator

Choose a reason for hiding this comment

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

If we want to allow producers to define their own extensions, do we need value to be more generic to support extensions with values of types other than CodableConcepts?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure I've just kept the extension as its always been defined but could make it more generic ?

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.

3 participants