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

feat: add support for FHIR subscriptions #585

Merged
merged 7 commits into from
Mar 7, 2022

Conversation

carvantes
Copy link
Contributor

@carvantes carvantes commented Mar 3, 2022

Same as #573, but adapted to work on the SMART branch.

There is nothing specific to SMART in the Subscriptions implementation.

I squashed Most of the Subscriptions changes into a single commit since that made it easier to deal with conflicts between develop and smart-develop

  • feat: add support for FHIR subscriptions
  • test: add enableSubscriptions flag on deployment
  • perf: partial failures for restHook Lambda (perf: partial failures for restHook Lambda #579)
  • chore: bump fwoa dependencies
  • chore: fix tests and lint issues
  • downgrade escape-string-regex to fix build

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

@awslabs awslabs deleted a comment from github-actions bot Mar 3, 2022
@awslabs awslabs deleted a comment from github-actions bot Mar 3, 2022
@awslabs awslabs deleted a comment from github-actions bot Mar 3, 2022
@awslabs awslabs deleted a comment from github-actions bot Mar 3, 2022
@awslabs awslabs deleted a comment from github-actions bot Mar 3, 2022
@awslabs awslabs deleted a comment from github-actions bot Mar 3, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #585 (9ea4906) into smart-develop (c91e731) will decrease coverage by 4.40%.
The diff coverage is 93.49%.

Impacted file tree graph

@@                Coverage Diff                @@
##           smart-develop     #585      +/-   ##
=================================================
- Coverage          94.84%   90.44%   -4.41%     
=================================================
  Files                  1        8       +7     
  Lines                 97      293     +196     
  Branches              14       58      +44     
=================================================
+ Hits                  92      265     +173     
- Misses                 5       27      +22     
- Partials               0        1       +1     
Impacted Files Coverage Δ
src/config.ts 78.18% <66.66%> (ø)
...c/subscriptions/reaperLambda/subscriptionReaper.ts 88.23% <88.23%> (ø)
src/subscriptions/restHookLambda/allowListUtil.ts 95.12% <95.12%> (ø)
src/index.ts 64.70% <100.00%> (ø)
src/subscriptions/restHookLambda/restHook.ts 100.00% <100.00%> (ø)
src/implementationGuides/loadCompiledIGs.ts 90.00% <0.00%> (ø)
src/authZConfig.ts 100.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c91e731...9ea4906. Read the comment docs.

@carvantes carvantes marked this pull request as ready for review March 3, 2022 19:24
ssvegaraju
ssvegaraju previously approved these changes Mar 3, 2022
@github-actions
Copy link

github-actions bot commented Mar 7, 2022

'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.’

@carvantes carvantes requested a review from ssvegaraju March 7, 2022 16:22
@carvantes carvantes merged commit 3ed101b into smart-develop Mar 7, 2022
@carvantes carvantes deleted the smart-feat-subscriptions branch March 7, 2022 16:35
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.

4 participants