Skip to content

feature/task-2 #548

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

krakz999
Copy link

What was done?

  • Implemented AWS infrastructure using CDK
  • Added convenience npm scripts for CDK deployment workflow
  • Updated Material UI theme configuration to use dark mode

"test": "jest",
"cdk": "cdk",
"cdk:synth": "npm run build && cdk synth",
"cdk:deploy": "npm run build && cdk deploy"
Copy link
Author

@krakz999 krakz999 Apr 11, 2025

Choose a reason for hiding this comment

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

The cdk deploy command will perform synthesis itself before initiating deployment, we don't have to run it separately for deployment purpose.

The CloudFront distribution will be invalidated automatically because the BucketDeployment construct includes the distributionPaths property, eliminating the need for cache invalidation in the npm script.

Copy link

@duncannevin duncannevin left a comment

Choose a reason for hiding this comment

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

@krakz999 This looks really good, nice work!

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.

3 participants