Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for update and create input field type known model types filtering #7929

Merged
merged 10 commits into from
Aug 25, 2021
Merged

Fix for update and create input field type known model types filtering #7929

merged 10 commits into from
Aug 25, 2021

Conversation

lazpavel
Copy link
Contributor

Description of changes

  • fixed update and create input field filter for known model types
  • added unit test

Description of how you validated changes

  • ran unit tests and added a new one

Checklist

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

@lazpavel lazpavel requested a review from a team as a code owner August 10, 2021 20:27
@cjihrig cjihrig self-assigned this Aug 10, 2021
@lazpavel lazpavel changed the base branch from master to 2493-clear-require-cache-between-lambda-invocations August 16, 2021 19:47
@lazpavel lazpavel changed the base branch from 2493-clear-require-cache-between-lambda-invocations to master August 16, 2021 19:47
@lazpavel lazpavel changed the base branch from master to 2493-clear-require-cache-between-lambda-invocations August 16, 2021 20:58
@lazpavel lazpavel changed the base branch from 2493-clear-require-cache-between-lambda-invocations to master August 16, 2021 20:58
@cjihrig
Copy link
Contributor

cjihrig commented Aug 25, 2021

@lazpavel this has a merge conflict.

@lazpavel lazpavel changed the base branch from master to 2493-clear-require-cache-between-lambda-invocations August 25, 2021 16:27
@lazpavel lazpavel changed the base branch from 2493-clear-require-cache-between-lambda-invocations to master August 25, 2021 16:28
@cjihrig cjihrig merged commit 16334f7 into aws-amplify:master Aug 25, 2021
jhockett pushed a commit that referenced this pull request Aug 25, 2021
…7929)

* fix(graphql-model-transformer): fixed update and create input field type known model types filtering

* fix(graphql-model-transformer): fixed update and create input field type known model types filtering

* chore(graphql-model-transformer): fixed merge conflicts

* fix(graphql-model-transformer): fixed update and create input field type known model types filtering

* fix(graphql-model-transformer): fixed update and create input field type known model types filtering

* chore(graphql-model-transformer): fixed merge conflicts
@github-actions github-actions bot added the referenced-in-release Issues referenced in a published release changelog label Sep 2, 2021
@github-actions
Copy link

github-actions bot commented Sep 2, 2021

👋 Hi, this pull request was referenced in the v5.4.0 release!

Check out the release notes here https://github.com/aws-amplify/amplify-cli/releases/tag/v5.4.0.

marcvberg pushed a commit to marcvberg/amplify-cli that referenced this pull request Oct 13, 2021
…ws-amplify#7929)

* fix(graphql-model-transformer): fixed update and create input field type known model types filtering

* fix(graphql-model-transformer): fixed update and create input field type known model types filtering

* chore(graphql-model-transformer): fixed merge conflicts

* fix(graphql-model-transformer): fixed update and create input field type known model types filtering

* fix(graphql-model-transformer): fixed update and create input field type known model types filtering

* chore(graphql-model-transformer): fixed merge conflicts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
referenced-in-release Issues referenced in a published release changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants