Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Recnet-api backend service initilization #151

Merged
merged 20 commits into from
Feb 29, 2024
Merged

Conversation

joannechen1223
Copy link
Collaborator

@joannechen1223 joannechen1223 commented Feb 24, 2024

Description

Create a project recnet-api as a backend service for recnet. The project is built by NestJS under nx. In this initial PR, I created a GET /health endpoint for testing purpose.

The code should be auto-deployed to AWS Elastic Beanstalk through the Codepipe line once merged to the master branch.

Related Issue

Create Nestjs backend and deploy to AWS

Notes

  • Dev Flow and CI
    After discussing with @swh00tw , we decided to maintain a recnet-api-dev branch for merging dev environment. Once a new feature PR is merged, it will automatically deploy to our dev environment. If everything is good to go, I will create another PR to merge recnet-api-dev branch to merge to master.

  • Code build issue investigation
    TL, DR pnpm installs node_modules with symlinks, so we'll need to add enable-symlink in the build spec.
    AWS deployment issue

TODO

  • Employ recnet.io subdomain
  • Change Codepipeline configuration to deploy recnet-api-dev branch

Copy link

vercel bot commented Feb 24, 2024

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

Name Status Preview Comments Updated (UTC)
rec-net ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 24, 2024 7:36am

fix: shuffle users in search page and all-users page
@joannechen1223 joannechen1223 changed the base branch from master to recnet-api-dev February 24, 2024 07:10
Copy link
Collaborator

@swh00tw swh00tw left a comment

Choose a reason for hiding this comment

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

LGTM, let's merge it and start API dev 🚀

Copy link
Collaborator

Choose a reason for hiding this comment

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

If we want to separate different prettier configs for different projects, I would suggest you move the .prettierrc config file to the root of recnet-api project.

@joannechen1223 joannechen1223 merged commit 00b4299 into recnet-api-dev Feb 29, 2024
3 checks passed
@swh00tw swh00tw deleted the init-backend branch January 9, 2025 03:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants