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

에러 해결, 필요없는 코드 정리 #55

Merged
merged 20 commits into from
Jan 21, 2025
Merged

Conversation

summersummerwhy
Copy link
Collaborator

@summersummerwhy summersummerwhy commented Jan 21, 2025

🔖 연관된 이슈

📂 작업 내용

  • bug fix 1: 도커 postgres 컨테이너에서 healthCheck가 안돼 계속해서 에러메세지 뜨는 문제 해결
  • bug fix 2: vite 버전 업데이트 되면서
    Blocked request. This host ("octodocs.local") is not allowed. To allow this host, add "octodocs.local" to "server.allowedHosts" in vite.config.js. 에러메세지 뜨는 문제 해결
  • bug fix 3:
      1. build
      • yarn build 과정에서 vite 모듈에 .env가 필요해짐, 필요한 환경변수 secrets에 등록후 frontend 디렉토리에 추가
      1. test
      • 이상하게 jest 모듈이 없어서 test가 실행안되는 에러가 계속 생김
      • 일단 test 전 yarn install하는 걸로 해결, but 시간이 너무 걸린다! 설치에만 Done in 112.78s.
      • test에만 필요한 모듈들을 src/backend/test/package.json안에 따로 정의, 해당 모듈들만 yarn install 후 node_modules를 src/backend로 이동시킴
      • Done in 14.05s. -> 9배 정도 단축 성공
  • chore 1: yarn reinstall로 모든 node_modules 디렉토리 제거 -> yarn.lock 제거 -> yarn install 순서대로 실행되게 package.json에 단축어 등록
  • chore 2: 컨트롤러, 서비스 코드 정리, 테스트 코드 정리, 정상적으로 작동하는 것 확인

📑 참고 자료 & 스크린샷 (선택)

📢 리뷰 요구사항 (선택)

@summersummerwhy summersummerwhy added 🐛 Bug 무언가 정상적으로 동작하지 않는 경우 🐧🚀😶‍🌫️ BE 백엔드 관련 이슈/PR 추적 라벨 ♻️ Refactor 기능 추가 외 코드 변경, 코드 품질과 관련된 경우 labels Jan 21, 2025
@summersummerwhy summersummerwhy self-assigned this Jan 21, 2025
@summersummerwhy summersummerwhy marked this pull request as draft January 21, 2025 07:31
@github-actions github-actions bot requested a review from ezcolin2 January 21, 2025 07:32
@summersummerwhy summersummerwhy marked this pull request as ready for review January 21, 2025 11:06
Copy link
Collaborator

@ezcolin2 ezcolin2 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@github-actions github-actions bot merged commit bc7a8f9 into develop Jan 21, 2025
7 checks passed
@github-actions github-actions bot deleted the refactor-be-#39 branch January 21, 2025 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐧🚀😶‍🌫️ BE 백엔드 관련 이슈/PR 추적 라벨 🐛 Bug 무언가 정상적으로 동작하지 않는 경우 ♻️ Refactor 기능 추가 외 코드 변경, 코드 품질과 관련된 경우
Projects
None yet
2 participants