fix: remove early removal of field from model in the case of connection

alharris-at · alharris-at · commit 3de34f4f0034 · 2022-01-07T10:49:38.000-08:00
diff --git a/packages/appsync-modelgen-plugin/src/visitors/appsync-visitor.ts b/packages/appsync-modelgen-plugin/src/visitors/appsync-visitor.ts
@@ -701,9 +701,6 @@ export class AppSyncModelVisitor<
               isList: false,
               isNullable: field.isNullable,
             });
-          } else if (connectionInfo.targetName !== 'id') {
-            // Need to remove the field that is targetName
-            removeFieldFromModel(model, connectionInfo.targetName);
           }
           field.connectionInfo = connectionInfo;
         }

Original file line number	Diff line number	Diff line change
`@@ -701,9 +701,6 @@ export class AppSyncModelVisitor<`
`701`	`701`	`isList: false,`
`702`	`702`	`isNullable: field.isNullable,`
`703`	`703`	`});`
`704`		`- } else if (connectionInfo.targetName !== 'id') {`
`705`		`- // Need to remove the field that is targetName`
`706`		`- removeFieldFromModel(model, connectionInfo.targetName);`
`707`	`704`	`}`
`708`	`705`	`field.connectionInfo = connectionInfo;`
`709`	`706`	`}`