Skip to content
This repository has been archived by the owner on Apr 13, 2023. It is now read-only.

chore: Merge feat-igs into develop-smart #259

Merged
merged 39 commits into from
Mar 23, 2021

Conversation

nguyen102
Copy link
Contributor

Issue #, if available:

Description of changes:
Combining the changes from feat-igs with develop-smart so that our SMART FHIR Server now support IGs.

Plan is to merge the code into feat-smart-igs first, and then later on merge feat-smart-igs into develop-smart when we're ready to release the code.

Code has been deployed to my personal SMART dev server successfully. Inferno and custom integration tests passed.

Note this PR does not include files under the directory javaHapiValidatorLambda/src/test/. I didn't include those files, as it would make this PR much harder to review. There are hundred of files under that directory and we will copy them over as is to feat-smart-igs.

Checklist:

  • Have you successfully deployed to an AWS account with your changes?
  • Have you written new tests for your core changes, as applicable?

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

rsmayda and others added 30 commits January 29, 2021 13:11
- Update dependencies to the newest
- Update auditLog dependencies to match the base package
- Add `${AWS::Region}` to IAM log policy
- Reduce access for the FHIR Server to just `GET`, `HEAD`, and `POST` requests (posts are needed as searches are POSTed to ES Domain)
- Add log retention and log encryption
- interface: add implementation guides compile interface
- persistence: match on resourceType as well as id when executing Read/VRead/DeleteVersionedRes operations
- persistence: resolve vid and meta attribute mismatch on concurrent Update requests
- parse xpath expressions to support choice of data types
- properly handle special characters in queries
* chore: add PR size labeler

* chore: update labeler yml syntax

* empty commit fix mainline protection

Co-authored-by: zheyanyu <[email protected]>
Co-authored-by: Tim Nguyen <[email protected]>
@github-actions
Copy link

'This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.’

@nguyen102 nguyen102 merged commit 2539034 into feat-smart-igs Mar 23, 2021
@nguyen102 nguyen102 deleted the feat-igs-with-profile-no-hapi-resources branch March 23, 2021 20:04
carvantes added a commit that referenced this pull request Mar 29, 2021
* chore: Merge feat-igs into develop-smart (#259)
* Add hapiValidator test files
* chore: Additional igs updates (#260)
* feat: support Implementation Guides

Co-authored-by: justinusmenzel <[email protected]>
Co-authored-by: Emil Diaz <[email protected]>
Co-authored-by: Tim Nguyen <[email protected]>
Co-authored-by: shyogesh-sw <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants