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

183-Create a subscription adapter for aws lambda #202

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

183-Create a subscription adapter for aws lambda #202

wants to merge 1 commit into from

Conversation

jadiaheno
Copy link
Contributor

@jadiaheno jadiaheno commented May 17, 2023 •
edited
Loading

What is this about

This PR adds an AWS Lambda FHIR subscription handler. This handler is in the subscriptions-lambda package.
A sample using this lambda was also provided.

Issue ticket numbers

How can this be tested?

Run the sample lambda api and register a communicationRequest: yarn dev

Known limitations/edge cases

NA

@jadiaheno jadiaheno force-pushed the 183-create-a-subscription-adapter-for-aws-lambda branch 2 times, most recently from b010985 to ba57935 Compare May 25, 2023 22:20
@jadiaheno jadiaheno marked this pull request as ready for review May 25, 2023 22:23
@jadiaheno jadiaheno requested a review from julienblin as a code owner May 25, 2023 22:23
@jadiaheno jadiaheno linked an issue May 25, 2023 that may be closed by this pull request
Create a subscription adapter for AWS Lambda #183
Open
@jadiaheno jadiaheno force-pushed the 183-create-a-subscription-adapter-for-aws-lambda branch from 35cccbb to 442fabb Compare May 30, 2023 22:45
@codecov
Copy link

codecov bot commented May 31, 2023 •
edited
Loading

Codecov Report

❗ No coverage uploaded for pull request base (main@621a7d8). Click here to learn what that means.
Patch coverage: 80.76% of modified lines in pull request are covered.

❗ Current head 62b817b differs from pull request most recent head dfa3c55. Consider uploading reports for the commit dfa3c55 to get more accurate results

Additional details and impacted files 
@@           Coverage Diff           @@
##             main     #202   +/-   ##
=======================================
  Coverage        ?   80.76%           
=======================================
  Files           ?        4           
  Lines           ?       52           
  Branches        ?        5           
=======================================
  Hits            ?       42           
  Misses          ?        8           
  Partials        ?        2
Impacted Files Coverage Δ
samples/sample-api-lambda/tests/nock.ts 61.53% <61.53%> (ø)
...es/sample-api-lambda/src/communication-requests.ts 100.00% <100.00%> (ø)
samples/sample-api-lambda/src/fhir.ts 100.00% <100.00%> (ø)
samples/sample-api-lambda/tests/fetch-polyfill.ts 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@jadiaheno jadiaheno force-pushed the 183-create-a-subscription-adapter-for-aws-lambda branch from 62b817b to dfa3c55 Compare May 31, 2023 06:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@julienblin julienblin Awaiting requested review from julienblin julienblin is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@julienblin julienblin

@jadiaheno jadiaheno

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create a subscription adapter for AWS Lambda
2 participants
@jadiaheno @julienblin