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

Fix problem with _include=MedicationRequest:medication #836

Merged
merged 1 commit into from May 22, 2018

Conversation

Projects
None yet
3 participants
@RuthAlk

RuthAlk commented Jan 24, 2018

A query for MedicationRequest that should include medication
did not work, because the SearchParamDefinition defined the
path as MedicationRequest.medication.as(Reference)

Enhanced the matchesInclude method to allow extra parts after
the search path

Ruth Alkema
Fix problem with _include=MedicationRequest:medication
A query for MedicationRequest that should include medication
did not work, because the SearchParamDefinition defined the
path as MedicationRequest.medication.as(Reference)
@coveralls

This comment has been minimized.

coveralls commented Jan 24, 2018

Coverage Status

Coverage increased (+0.008%) to 72.472% when pulling 9a4933f on RuthAlk:include_patch into 0677f35 on jamesagnew:master.

@jamesagnew jamesagnew merged commit 3a68bc1 into jamesagnew:master May 22, 2018

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.008%) to 72.472%
Details

jamesagnew added a commit that referenced this pull request May 22, 2018

@jamesagnew

This comment has been minimized.

Owner

jamesagnew commented May 22, 2018

Thanks for the PR! It has been merged.

Please let me know if you would like your real name added to the credits

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment