Skip to content

4주차 미션 [루가]#4

Open
ochyeon wants to merge 3 commits intomainfrom
luga/week4
Open

4주차 미션 [루가]#4
ochyeon wants to merge 3 commits intomainfrom
luga/week4

Conversation

@ochyeon
Copy link
Copy Markdown
Collaborator

@ochyeon ochyeon commented Apr 8, 2026

🚩 관련 이슈

  • 없음

📌 구현 결과

  • 기존 ERD 내 미션 상태 ENUM 수정
  • DB_USER, DB_PW, DB_URL : 환경 변수로 설정
  • user, mission, review를 메인 도메인으로 설정, 각 요소들의 기존 구조 작성
  • SwaggerConfig 작성 : 워크북 참고

❓ 리뷰 요청

  • 기존 ERD에서 미션 상태 ENUM 외에는 특별히 수정하지 않았는데, 추가로 수정하면 좋을 부분이 있을까요?
  • 각 도메인 폴더 내 파일들 중 불필요하거나 혹은 필요한데 생략된 부분이 있을까요?

🤔 질문

💬 기타 공유 사항

@ochyeon ochyeon self-assigned this Apr 8, 2026
@ochyeon ochyeon changed the title 1주차 미션 [루가] 4주차 미션 [루가] Apr 8, 2026
@yujining3827
Copy link
Copy Markdown
Collaborator

4주차도 수고하셨습니다 ~!!

Copy link
Copy Markdown
Member

@yangjiae12 yangjiae12 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 리드미에 적혀있는 것처럼 luga 폴더 안에 프로젝트가 위치하도록 수정하셔야 합니다. 그리고 코멘트 달은 것들만 수정하면 좋을 것 같아요~

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Restaurant는 mission 도메인에 넣기보단 패키지를 따로 두면 좋을 것 같습니다!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

미션 상태 외에도 유저 상태라든지 status 이넘값이 많이 생길 수 있으니 단순 status라고 쓰지 않고 MissionStatus 같은 식으로 어떤 것의 상태인지 명시해 주는 것이 좋습니다.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

첫 글자는 대문자로 써주세요! 이 외에도 유저 도메인에 많은 파일들이 소문자로 되어 있어서 같이 고쳐주세요.

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