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 inCompartment for a collection of owners #806

Merged
merged 2 commits into from Mar 19, 2018

Conversation

Projects
None yet
2 participants
@JiajingLiang
Contributor

JiajingLiang commented Dec 14, 2017

concerning issue: #778

Jiajing LIANG added some commits Dec 13, 2017

Jiajing LIANG
@JiajingLiang

This comment has been minimized.

Contributor

JiajingLiang commented Dec 21, 2017

Added some test to ensure that inCompartment with a collection of owners works.
But I don't understand why we have exactly the some tests for Dstu3 and R4. I mean why separate the tests according to fhir context version? @jamesagnew

@jamesagnew

This comment has been minimized.

Owner

jamesagnew commented Mar 19, 2018

Looks great, merging now!

@jamesagnew jamesagnew merged commit e9ee872 into jamesagnew:master Mar 19, 2018

0 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
continuous-integration/appveyor/pr AppVeyor build failed
Details

jamesagnew added a commit that referenced this pull request Mar 19, 2018

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