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

ISOM-1293: Add CodeBuild logic #382

Merged
merged 20 commits into from
Aug 13, 2024
Merged

ISOM-1293: Add CodeBuild logic #382

merged 20 commits into from
Aug 13, 2024

Conversation

harishv7
Copy link
Contributor

@harishv7 harishv7 commented Jul 28, 2024

TL;DR

Added a new field to the Site table to store the AWS CodeBuild Project ID and integrated AWS CodeBuild into the publishing process.

What changed?

  • Updated package.json and package-lock.json to include @aws-sdk/client-codebuild and @aws-sdk/smithy-client
  • Modified Prisma schema and generated types to include codeBuildId field
  • Created a new migration to add codeBuildId column to the Site table
  • Added new AWS CodeBuild service module (codebuild.service.ts)
  • Updated page.router.ts to initiate a CodeBuild project run on page publish
  • Enhanced error logging in PublishButton.tsx

How to test?

  1. Run the migration to add the codeBuildId column: prisma migrate deploy
  2. Add a valid CodeBuild Project ID to one of the Site records
  3. Attempt to publish a page and verify that a CodeBuild project run is initiated

Why make this change?

This change integrates AWS CodeBuild into the page publishing workflow, allowing for automated builds and deployments whenever a page is published.


TL;DR

This PR integrates AWS CodeBuild into the project and introduces new site configurations in the Prisma schema.

What changed?

  • Added new dependencies: @aws-sdk/client-codebuild, @aws-sdk/smithy-client
  • Updated Prisma schema to include shortName and codeBuildId in the Site model
  • Created new AWS CodeBuild service for managing CodeBuild projects
  • Updated seed.ts to include shortName for seed data
  • Enhanced page.router.ts to initiate CodeBuild upon page publish
  • Added new functions getSiteNameAndCodeBuildId and setSiteCodeBuildId in site.service.ts

How to test?

  1. Run the Prisma migration to update the schema.
  2. Seed the database with new data.
  3. Publish a page and check if an AWS CodeBuild project is initiated.
  4. Verify the codeBuildId is correctly set in the Site model.

Why make this change?

The integration with AWS CodeBuild allows us to automate the build and deployment process, making the system more efficient and reliable. The new site configurations provide better structure and flexibility for site management.

Copy link

vercel bot commented Jul 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
isomer-studio ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 12, 2024 5:54pm

Copy link
Contributor Author

harishv7 commented Jul 28, 2024

@harishv7 harishv7 changed the title add codebuild logic ISOM-1293: Add CodeBuild logic Jul 28, 2024
Copy link

linear bot commented Jul 28, 2024

@harishv7 harishv7 marked this pull request as ready for review July 28, 2024 15:46
@harishv7 harishv7 requested a review from a team as a code owner July 28, 2024 15:46
@harishv7 harishv7 marked this pull request as draft July 29, 2024 08:10
Copy link
Contributor Author

harishv7 commented Aug 12, 2024

Merge activity

  • Aug 12, 12:44 PM EDT: @harishv7 started a stack merge that includes this pull request via Graphite.
  • Aug 12, 12:47 PM EDT: Graphite couldn't merge this PR because it had conflicts with the trunk branch.

@harishv7 harishv7 changed the base branch from 07-28-add_publish_endpoint to graphite-base/382 August 12, 2024 16:44
@harishv7 harishv7 changed the base branch from graphite-base/382 to main August 12, 2024 16:45
@harishv7 harishv7 merged commit 0b4fc45 into main Aug 13, 2024
17 checks passed
@harishv7 harishv7 deleted the 07-28-add_codebuild_logic branch August 13, 2024 02:53
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.

2 participants