amplify-cli
amplify-cli copied to clipboard
feat: add AppSync Todo template for Nodejs
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
- adding AppSync Todo templates for IAM and API key based examples.
- added new prompt that inputs auth type.
- Modify lambda-function.ts file to move cursor based on auth type
- added new attribute to the core settings to allow auth selection
- 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.
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
Codecov Report
Merging #10841 (5c2961d) into dev (f0eb844) will increase coverage by
0.00%
. The diff coverage isn/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
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
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
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
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