amplify-cli icon indicating copy to clipboard operation
amplify-cli copied to clipboard

feat: add AppSync Todo template for Nodejs

Open ykethan opened this issue 2 years ago • 5 comments

Description of changes

DRAFT -add nodejs templates

The PR is to add Nodejs template examples that allow Lambda functions to call a AppSync API. Changes include

  1. adding AppSync Todo templates for IAM and API key based examples.
  2. added new prompt that inputs auth type.
  3. Modify lambda-function.ts file to move cursor based on auth type
  4. added new attribute to the core settings to allow auth selection
  5. add e2e tests for both the templates

Issue #, if available

Description of how you validated changes

Wrote e2e test and ran them locally. also used the amplify-dev in a frontend app to test the flow.

Checklist

  • [x] PR description included
  • [ ] yarn test passes
  • [x] Tests are changed or added
  • [ ] Relevant documentation is changed or added (and PR referenced)
  • [ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies

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

ykethan avatar Aug 03 '22 22:08 ykethan

This pull request introduces 3 alerts when merging 45552099bc8e4b0f12641af906b8ce51c78026e2 into 2b17187b42cc2352993259e20d39226a227ee200 - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

lgtm-com[bot] avatar Aug 03 '22 22:08 lgtm-com[bot]

Codecov Report

Merging #10841 (5c2961d) into dev (f0eb844) will increase coverage by 0.00%. The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev   #10841   +/-   ##
=======================================
  Coverage   47.32%   47.32%           
=======================================
  Files         674      674           
  Lines       33212    33212           
  Branches     6698     6698           
=======================================
+ Hits        15716    15717    +1     
+ Misses      15812    15811    -1     
  Partials     1684     1684           
Impacted Files Coverage Δ
...li/src/domain/amplify-usageData/getUsageDataUrl.ts 100.00% <0.00%> (+12.50%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

codecov-commenter avatar Aug 03 '22 22:08 codecov-commenter

This pull request introduces 3 alerts when merging 36eaa20649bf24f34bca6910926cd1dccd1a5496 into 2b17187b42cc2352993259e20d39226a227ee200 - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

lgtm-com[bot] avatar Aug 04 '22 23:08 lgtm-com[bot]

This pull request introduces 3 alerts when merging 13ff9b037e3be0fd5e4e600353784091de7bf76d into 536fdf2fd2ed20e196656755033b5b2ce9f68fa0 - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

lgtm-com[bot] avatar Aug 06 '22 03:08 lgtm-com[bot]

This pull request introduces 3 alerts when merging dd5c97cc22ef27336b98b7764497c822942b879d into 536fdf2fd2ed20e196656755033b5b2ce9f68fa0 - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

lgtm-com[bot] avatar Aug 06 '22 04:08 lgtm-com[bot]

This pull request introduces 1 alert when merging 787a1718ad63c5e4166afb98f3cb860c73776f36 into f0eb84472ac76d7b632b9716b43185ce0eaee3ce - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

lgtm-com[bot] avatar Aug 24 '22 21:08 lgtm-com[bot]