From d5fe9236704c539c6dd4e262efb3c07358545baa Mon Sep 17 00:00:00 2001 From: Nicholas DeJaco <54116900+ndejaco2@users.noreply.github.com> Date: Wed, 25 Oct 2023 14:38:20 -0700 Subject: [PATCH] chore(appsync): Tidy up README for Merged APIs (#27679) * This change just updates the README to remove an unnecessary use of lazy and show the second api being associated with AUTO_MERGE mode which had been removed by mistake. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license* --- packages/aws-cdk-lib/aws-appsync/README.md | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/packages/aws-cdk-lib/aws-appsync/README.md b/packages/aws-cdk-lib/aws-appsync/README.md index ffae1b3f8ac20..637dbb230edb2 100644 --- a/packages/aws-cdk-lib/aws-appsync/README.md +++ b/packages/aws-cdk-lib/aws-appsync/README.md @@ -377,18 +377,12 @@ const firstApi = new appsync.GraphqlApi(this, 'FirstSourceAPI', { name: 'FirstSourceAPI', definition: appsync.Definition.fromFile(path.join(__dirname, 'appsync.merged-api-1.graphql')), }); -firstApi.addNoneDataSource('FirstSourceDS', { - name: cdk.Lazy.string({ produce(): string { return 'FirstSourceDS'; } }), -}); // second source API const secondApi = new appsync.GraphqlApi(this, 'SecondSourceAPI', { name: 'SecondSourceAPI', definition: appsync.Definition.fromFile(path.join(__dirname, 'appsync.merged-api-2.graphql')), }); -secondApi.addNoneDataSource('SecondSourceDS', { - name: cdk.Lazy.string({ produce(): string { return 'SecondSourceDS'; } }), -}); // Merged API const mergedApi = new appsync.GraphqlApi(this, 'MergedAPI', { @@ -399,6 +393,10 @@ const mergedApi = new appsync.GraphqlApi(this, 'MergedAPI', { sourceApi: firstApi, mergeType: appsync.MergeType.MANUAL_MERGE, }, + { + sourceApi: secondApi, + mergeType: appsync.MergeType.AUTO_MERGE, + } ], }), });