-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
@aws-cdk\aws-amplify-alpha: branch #28289
Comments
Thanks for the report. Looks like we should add the dependency. |
…ndlers (#28373) Add dependency from from **@aws-cdk/custom-resource-handlers** to **@aws-cdk/aws-amplify-alpha** as part of effort to standardize custom resource creation and bundling of source code. Verified addition with `yarn install` and `yarn test`. Closes #28289. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
|
The issue is not fixed in version 2.118.0a0 There is still an error when running "amplify_app.add_branch" " Cannot find entry file at C:\Users\user1\AppData\Local\Temp\jsii-kernel-1N88ev\node_modules@aws-cdk\custom-resource-handlers\dist\aws-amplify-alpha\asset-deployment-handler\index.js" |
Describe the bug
When running stack with the following code (Python) it is not possible to deploy to AWS Amplify from asset as it is described in documentation
The code
The reason is reference in branch.ts on line 236
There is no dependency from @aws-cdk/custom-resource-handlers package inside @aws-cdk/aws-amplify-alpha
Expected Behavior
Amplify branch should be created from an asset
Current Behavior
Amplify branch is not created from an asset
The following error appears
Reproduction Steps
Execute the code
Possible Solution
Add dependency to @aws-cdk/custom-resource-handlers package inside @aws-cdk/aws-amplify-alpha and publish @aws-cdk/custom-resource-handlers itself
Additional Information/Context
No response
CDK CLI Version
aws-cdk.aws-amplify-alpha==2.114.1a0
Framework Version
No response
Node.js Version
18.14.0
OS
Windows 11 Pro
Language
Python
Language Version
3.10.11
Other information
No response
The text was updated successfully, but these errors were encountered: