Skip to content

Conversation

@dpilch
Copy link
Contributor

@dpilch dpilch commented Feb 14, 2025

Description of changes

  • Remove experimental flags
  • Remove temporary testing setup
    • No longer needed because the feature flag is released in gen 1
CDK / CloudFormation Parameters Changed

No changes other than removing experimental tags from features.

Issue #, if available

N/A

Description of how you validated changes

E2E tests: https://us-east-1.console.aws.amazon.com/codesuite/codebuild/594813022831/projects/amplify-category-api-e2e-workflow/batch/amplify-category-api-e2e-workflow:8c9ce3b2-4ba0-49cb-b546-8f5fbeef9415?region=us-east-1

Checklist

  • PR description included
  • yarn test passes
  • E2E test run linked
  • Tests are changed or added

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

@dpilch dpilch force-pushed the migrations-release branch from a4885d6 to 1f91d69 Compare March 10, 2025 14:45
// start block
// deletion protection is enabled for migrated tables
// disable deletion protection to teardown the tests
await Promise.allSettled(
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We actually need this block because the tables will still have deletion protection when tearing down the test.

@dpilch dpilch marked this pull request as ready for review March 12, 2025 15:03
@dpilch dpilch requested a review from a team as a code owner March 12, 2025 15:03
tejas2008
tejas2008 previously approved these changes Mar 12, 2025
@dpilch dpilch force-pushed the migrations-release branch from a87baa1 to 8981707 Compare April 11, 2025 16:29
@dpilch dpilch enabled auto-merge (squash) April 11, 2025 16:33
@dpilch dpilch merged commit 97f9ccb into main Apr 14, 2025
7 of 8 checks passed
@dpilch dpilch deleted the migrations-release branch April 14, 2025 15:48
dpilch pushed a commit that referenced this pull request Apr 17, 2025
* chore: update .jsii assembly

* chore: update .jsii assembly

* chore: upgrade to Node 20 (#3258)

* feat: add grantStreamRead to amplify managed table (#3253)

* chore: prep migrations release (#3169)

---------

Co-authored-by: amplify-data-ci <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants