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

Invalid null value for display #2740

Closed
codyebberson opened this issue Aug 28, 2023 · 1 comment · Fixed by #2746
Closed

Invalid null value for display #2740

codyebberson opened this issue Aug 28, 2023 · 1 comment · Fixed by #2746
Assignees
@ThatOneBro
Labels
bug Something isn't working fhir-datastore Related to the FHIR datastore, includes API and FHIR operations
Milestone
August 31, 2023

Comments

@codyebberson
Copy link
Member

Found on staging - likely a bug in CodeableConceptInput

image

Thanks @ThatOneBro
@reshmakh reshmakh added the react Features, fixes and enhancements to react component library label Aug 28, 2023
@ThatOneBro ThatOneBro self-assigned this Aug 28, 2023
@ThatOneBro ThatOneBro added this to the August 31, 2023 milestone Aug 28, 2023
@ThatOneBro ThatOneBro mentioned this issue Aug 29, 2023
Merged
@ThatOneBro ThatOneBro added fhir-datastore Related to the FHIR datastore, includes API and FHIR operations and removed react Features, fixes and enhancements to react component library labels Aug 29, 2023
@ThatOneBro
Copy link
Member

Turned out to be a bug in the server rather than the component. Basically unfiltered null values were sneaking in from the ValueSet-expand queries since the server was basically not filtering them for this specific case.

@ThatOneBro ThatOneBro added the bug Something isn't working label Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ThatOneBro ThatOneBro

Labels
bug Something isn't working fhir-datastore Related to the FHIR datastore, includes API and FHIR operations
Projects
Medplum Product Roadmap
Archived in project
Milestone
August 31, 2023
Development

Successfully merging a pull request may close this issue.

fix(ValueSet-expand): delete null display fields before resp medplum/medplum
3 participants
@codyebberson @ThatOneBro @reshmakh