Merged
Conversation
munwalk
approved these changes
Mar 23, 2026
Collaborator
munwalk
left a comment
There was a problem hiding this comment.
1주차 과제 수고하셨습니다!
키워드를 깔끔하게 정리하셨네요! 각 개념의 특징과 장단점을 명확하게 구분해서 작성하신 점이 좋습니다.
member 테이블에서 soft delete 방식(status, inactive_date)을 도입하신 점이 훌륭합니다. 회원 탈퇴 후에도 작성한 리뷰나 미션 이력을 유지할 수 있어서 데이터 무결성을 지킬 수 있다는 장점이 있습니다~
특히 review 테이블을 member_mission 기준으로 연결하신 설계가 매우 논리적입니다. 리뷰는 미션 수행 후 작성되는 것이니 이 구조가 더 명확하네요! 그리고 review_image를 별도 테이블로 분리해서 여러 장의 사진을 첨부할 수 있도록 1:N 관계로 설계하신 점도 좋습니다.
수고하셨습니다!
첨언) 다음 과제부터는 노션 링크를 제출해주시는게 아니라 keyword.md와 mission.md 파일과 같은 md파일로 노션 내 정리해주신 키워드와 미션 기록 및 미션 결과 등을 제출해주시는 게 좋을 것 같습니다! 잘 모르시겠다면 스터디장님이 올린 방식을 참고하셔도 좋을 듯 싶습니다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.