Skip to content

Conversation

McNaBry
Copy link

@McNaBry McNaBry commented Oct 29, 2024

Description

  • Created a Github Actions workflow to push frontend files to AWS S3 bucket
  • Created a Github Actions workflow to push docker images to AWS ECR

Checklist

  • I have updated documentation
  • All tests passing

Screenshots (if applicable)

Copy link

@samuelim01 samuelim01 left a comment

Choose a reason for hiding this comment

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

LGTM, shall await the merge of #52 and #68. Also I can't seem to add a new secret with destroying the environment, so you might have to reconfigure the environment variables.

* The build is triggered for a specific service only if its files have been changes
McNaBry and others added 2 commits October 31, 2024 13:51
* Remove trigger on pull request
* Add conditional trigger based on frontend directory
@McNaBry McNaBry requested a review from samuelim01 November 3, 2024 16:15
Copy link

@samuelim01 samuelim01 left a comment

Choose a reason for hiding this comment

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

Please standardise between backend_force.yml and frontend force.yml 🫡

@McNaBry McNaBry requested a review from samuelim01 November 3, 2024 16:27
Copy link

@samuelim01 samuelim01 left a comment

Choose a reason for hiding this comment

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

LGTM good work!

@McNaBry McNaBry merged commit ccd3d92 into main Nov 3, 2024
11 checks passed
@McNaBry McNaBry deleted the ci-cd-aws-deploy branch November 3, 2024 16:39
KhoonSun47 added a commit that referenced this pull request Nov 3, 2024
* main:
  Create GitHub Actions for auto deployment to AWS (#66)
  History Service (#69)
  Minor Fix To Handle isForfeit Status (#78)

# Conflicts:
#	services/collaboration/src/controllers/roomController.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants