fix(amplify-category-notifications): fix email channel removal
fix #5848
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Codecov Report
Merging #6030 (7b942fc) into master (db9bf58) will not change coverage. The diff coverage is
n/a.
@@ Coverage Diff @@
## master #6030 +/- ##
========================================
Coverage 58.21% 58.21%
========================================
Files 440 440
Lines 20160 20160
Branches 3988 3782 -206
========================================
Hits 11736 11736
- Misses 7609 7629 +20
+ Partials 815 795 -20
| Impacted Files | Coverage Δ | |
|---|---|---|
| ...mplify-category-notifications/lib/channel-Email.js | 0.00% <ø> (ø) |
|
| packages/amplify-util-mock/src/api/api.ts | 0.00% <0.00%> (ø) |
|
| packages/graphql-mapping-template/src/print.ts | 35.29% <0.00%> (ø) |
|
| packages/amplify-util-mock/src/storage/storage.ts | 0.00% <0.00%> (ø) |
|
| ...ges/amplify-util-mock/src/CFNParser/stack/index.ts | 0.00% <0.00%> (ø) |
|
| ...es/amplify-util-mock/src/api/resolver-overrides.ts | 0.00% <0.00%> (ø) |
|
| ...es/graphql-transformer-core/src/stripDirectives.ts | 35.29% <0.00%> (ø) |
|
| .../amplify-cli-core/src/state-manager/pathManager.ts | 67.85% <0.00%> (ø) |
|
| .../amplify-util-mock/src/utils/lambda/loadMinimal.ts | 0.00% <0.00%> (ø) |
|
| ...graphql-transformer-core/src/TransformerContext.ts | 25.09% <0.00%> (ø) |
|
| ... and 10 more |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update db9bf58...7b942fc. Read the comment docs.
@UnleashedMind I'd like to run the test suite on this, but do you mind rebasing on dev first? Thanks!
Closing due to inactivity.