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

Fixes issue #799 by adding logic to handle the R4 and DSTU2_1 contexts in ca.uhn.fhir.jaxrs.server.util.JaxRsRequest. #800

Merged
merged 2 commits into from Mar 19, 2018

Conversation

Projects
None yet
2 participants
@InfiniteLoop90
Contributor

InfiniteLoop90 commented Nov 28, 2017

Fixes issue #799 by adding logic to handle the R4 and DSTU2_1 contexts in ca.uhn.fhir.jaxrs.server.util.JaxRsRequest.

Also changed the locations in the hapi-fhir-jaxrsserver-base module where it was doing if-else branches based on the FhirVersionEnum to now use switch statements instead.

InfiniteLoop90 added some commits Nov 28, 2017

Fixes issue #799 by adding logic to handle the R4 and DSTU2_1 context…
…s in ca.uhn.fhir.jaxrs.server.util.JaxRsRequest.

Also changed the locations in the hapi-fhir-jaxrsserver-base module where it was doing if-else branches based on the FhirVersionEnum to now use switch statements instead.
Related to issue #799, adding DSTU2_1 and R4 contexts to the FhirCont…
…ext#(FhirVersionEnum, Collection<Class<? extends IBaseResource>>) method
@jamesagnew

This comment has been minimized.

Owner

jamesagnew commented Mar 19, 2018

Looks great, merging now!

@jamesagnew jamesagnew merged commit 142c4c6 into jamesagnew:master Mar 19, 2018

0 of 2 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI 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