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: allow api updates without migration #9864
Conversation
50828b2
to
46d52d9
Compare
This pull request introduces 1 alert when merging 46d52d9 into b22b67d - view on LGTM.com new alerts:
|
packages/amplify-category-api/src/provider-utils/awscloudformation/cfn-api-artifact-handler.ts
Outdated
Show resolved
Hide resolved
…tion/cfn-api-artifact-handler.ts Co-authored-by: John Hockett <jhockett@users.noreply.github.com>
This pull request introduces 1 alert when merging 5778213 into b22b67d - view on LGTM.com new alerts:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This pull request introduces 1 alert when merging 9a8873a into b22b67d - view on LGTM.com new alerts:
|
👋 Hi, this pull request was referenced in the v7.6.24 release! Check out the release notes here https://github.com/aws-amplify/amplify-cli/releases/tag/v7.6.24. |
Description of changes
When extensibility was implemented, api updates were blocked unless the resource was migrated. This change allows updates to the resource without requiring an update
Issue #, if available
fixes #9675
Description of how you validated changes
manually verified and added a migration e2e test
Checklist
yarn test
passesBy submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.