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.

Sign up for GitHub

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

Jump to bottom

Incorrect Validation on Recursive Elements #4299

Closed
rahul1 opened this issue Apr 2, 2024 · 1 comment · Fixed by #4319
Closed

Incorrect Validation on Recursive Elements #4299

rahul1 opened this issue Apr 2, 2024 · 1 comment · Fixed by #4319
Assignees
@codyebberson
Labels
bug Something isn't working
Milestone
April 30th, 2024

Comments

@rahul1
Copy link
Member

rahul1 commented Apr 2, 2024

User Driven: https://discord.com/channels/905144809105260605/1224559888785604778/1224559888785604778

Repro

Try to create a Consent resource where Consent.provision[0].provision is non-null

Expected Behavior

The resource should be saved successfully

Observed Behavior

Error as shown below:
image
@rahul1 rahul1 added the bug Something isn't working label Apr 2, 2024
@codyebberson codyebberson added this to the April 30th, 2024 milestone Apr 2, 2024
codyebberson added a commit that referenced this issue Apr 2, 2024
@codyebberson
Fixes #4299 - use correct cardinality on contentReference
a317757
github-merge-queue bot pushed a commit that referenced this issue Apr 3, 2024
@codyebberson
Fixes #4299 - use correct cardinality on contentReference (#4304)
41cbff1
@ThatOneBro
Copy link
Member

Closed as completed via #4304

medplumbot added a commit that referenced this issue Apr 4, 2024
@medplumbot
Release Version 3.1.1
58b5b00 
Added sortStringArray utility function (#4297)Remove 'dist' from eligibility demo (#4300)
Fixes #3944 - patient everything pagination (#4296)
Fixed fragment with only one child warning (#4306)
Fixes #4299 - use correct cardinality on contentReference (#4304)
Add "New..." action in Medplum Provider app (#4303)
Updating case study material (#4305)
Drop unused columns from lookup tables (#4294)
Fixes #3905 - better graphql mutation error reporting (#4308)
Fixes #4309 - AWS Parameter Store support for required database SSL (#4310)
@medplumbot medplumbot mentioned this issue Apr 4, 2024
Merged
codyebberson pushed a commit that referenced this issue Apr 4, 2024
@medplumbot @codyebberson
Release Version 3.1.1
bb6c7d7 
Added sortStringArray utility function (#4297)Remove 'dist' from eligibility demo (#4300)
Fixes #3944 - patient everything pagination (#4296)
Fixed fragment with only one child warning (#4306)
Fixes #4299 - use correct cardinality on contentReference (#4304)
Add "New..." action in Medplum Provider app (#4303)
Updating case study material (#4305)
Drop unused columns from lookup tables (#4294)
Fixes #3905 - better graphql mutation error reporting (#4308)
Fixes #4309 - AWS Parameter Store support for required database SSL (#4310)
github-merge-queue bot pushed a commit that referenced this issue Apr 5, 2024
@medplumbot
Release Version 3.1.1 (#4319)
5c643b5 
Added sortStringArray utility function (#4297)Remove 'dist' from eligibility demo (#4300)
Fixes #3944 - patient everything pagination (#4296)
Fixed fragment with only one child warning (#4306)
Fixes #4299 - use correct cardinality on contentReference (#4304)
Add "New..." action in Medplum Provider app (#4303)
Updating case study material (#4305)
Drop unused columns from lookup tables (#4294)
Fixes #3905 - better graphql mutation error reporting (#4308)
Fixes #4309 - AWS Parameter Store support for required database SSL (#4310)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@codyebberson codyebberson

Labels
bug Something isn't working
Projects
Medplum Product Roadmap
Archived in project
Milestone
April 30th, 2024
Development

Successfully merging a pull request may close this issue.

Release Version 3.1.1 medplum/medplum
3 participants
@codyebberson @rahul1 @ThatOneBro