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

Patch operation on OperationsWhichAllowPreferHeader #1022

Merged
merged 3 commits into from Aug 12, 2018

Conversation

Projects
None yet
2 participants
@alinleonard

alinleonard commented Jul 9, 2018

Based on https://www.hl7.org/fhir/http.html#2.21.0.5.2 . Added patch operation on OperationsWhichAllowPreferHeader so it will also work according to FHIR

@alinleonard

This comment has been minimized.

Show comment
Hide comment
@alinleonard

alinleonard Jul 10, 2018

Travis fail not related to this.
RestHookTestWithInterceptorRegisteredToDaoConfigR4Test.testRestHookSubscriptionXml:201->BaseJpaTest.waitForSize:363 Size 0 is != target 1 - Got: []

alinleonard commented Jul 10, 2018

Travis fail not related to this.
RestHookTestWithInterceptorRegisteredToDaoConfigR4Test.testRestHookSubscriptionXml:201->BaseJpaTest.waitForSize:363 Size 0 is != target 1 - Got: []

@jamesagnew jamesagnew merged commit 78a7d64 into jamesagnew:master Aug 12, 2018

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details

jamesagnew added a commit that referenced this pull request Aug 12, 2018

@jamesagnew

This comment has been minimized.

Show comment
Hide comment
@jamesagnew

jamesagnew Aug 12, 2018

Owner

Thanks for the PR, it has been merged! Please let me know if there are any specific details you'd like added in your developer profile in the pom.xml.

Owner

jamesagnew commented Aug 12, 2018

Thanks for the PR, it has been merged! Please let me know if there are any specific details you'd like added in your developer profile in the pom.xml.

@alinleonard

This comment has been minimized.

Show comment
Hide comment
@alinleonard

alinleonard Aug 24, 2018

Hello @jamesagnew

Thank you for your response. For the contributors section I would like you to add Cerner Corporation as organization.

alinleonard commented Aug 24, 2018

Hello @jamesagnew

Thank you for your response. For the contributors section I would like you to add Cerner Corporation as organization.

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