You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
키워드는 키워드 폴더에 주차폴더 생성 -> keyword.md 생성 후 작성 -> keyword: 로 시작하는 커밋메시지 컨벤션에 맞게 커밋
미션은 미션 폴더에 주차폴더 생성 -> mission.md 생성 후 작성 -> mission: 로 시작하는 커밋메시지 컨벤션에 맞게 커밋
제가 올려드린 워크북 제출 pdf 다시 확인하시고 다음 주차 부터는 형식에 맞게 업로드 해주세용. 혹시 잘 안되시면 디스코나 카톡주셔도 됩니다.
++ 0주차만 봐서 몰랐는데 폴더 구조 1주차에 수정하신거 확인 했습니다!
++ 추가로 사진을 따로 업로드 해주시고 링크를 걸어주신 것 같습니다. 이 방법이 편하시면 그대로 유지하셔도 되고 아니면 mission/chapter00 폴더에 첨부 후 해당 사진을 끌어서 .md 파일에 넣으면 사진까지 한번에 해당 레포에 첨부가 가능합니다.
ERD 논리값, 물리값 잘 분리해서 작성해주셨어요. NULL여부 표시되지 않은 테이블이 많아서 설정에서 NULL여부와 Comment등 추가해서 명시해주시면 좋을 것 같습니다.
약관 테이블에 버전까지 고려해주신 점 좋습니다.
정책 동의 여부에 대해 동의일을 저장하는걸로 보아 동의하는 경우만 저장하는 설계를 하신 것 같습니다. 좋은생각이에요!
정책 동의와 관련해서 현재 설계처럼 진행하게 될 경우 사용자가 선택약관에 대해 동의 -> 철회를 수정하게 되면 특정 시점에 동의였는지 철회였는지 체크가 불가능해요. 해당 워크북 진행동안에는 이정도까지 구체적인 구현이 필요하지는 않지만 이후 프로젝트를 고려해서 찾아보시면 좋을 것 같습니다.
음식 카테고리 관련해서 순서까지 고려해서 저장하신점 너무 좋아요.
지역에 완료보상 포인트를 넣어주셨는데 미션에도 보상 포인트가 있어서 지역에 완료 보상 포인트라는게 어떤 의미인지 잘 모르겠어요.
사용자에 deleted_at 을 통해 soft delete 고려해주시면 좋을 것 같습니다.
사용자 미션과 관련해 여러 일자에 대해 따로 저장하도록 설계해주셨습니다. 상태를 따로 저장하고 있기 때문에 각각의 시간을 두지 않고 통합하는 방법도 있을 것 같습니다. 혹은 상태 변경에 대한 이력을 저장하려고 하신거라면 별도 테이블을 생성하는게 좋을 것 같아요.
사용자의 포인트가 있으니 포인트 이력 (예) point_history 등이 있으면 좋을 것 같습니다.
미션에 대해서 만료 기간도 존재했던것 같은데 이 부분도 한 번 생각해봐 주세용.
리뷰가 사용자 미션에 연결되어 있습니다. 아마 리뷰가 미션에 연결되는 상황을 생각하신 것 같고 피그마에도 약간의 오해가 있을법하게 나와있는 것 같은데 일반적으로 봤을 때 가게에 리뷰를 다는 개념이 맞을 것 같아요.
문의와 관련된 테이블이 없는것 같아요.
가게의 영업시간과 관련된 값이 없는 것 같습니다. JSON 형태나 별도 테이블을 추가해 저장할 수 있을 것 같아요.
그린 0주차 수고하셨어요. 해당 ERD 설계를 가지고 이후 워크북을 진행하게 되는데 디테일한 설계는 따로 약간만 공부하셔도 되고 꼭 필요한 부분은 이후에 수정하려고 하면 귀찮아질 수 있어서 미리 수정해두시면 좋을 것 같습니다.
앞으로도 화이팅 해주세요!!
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
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.
✅ 워크북 체크리스트
✅ 컨벤션 체크리스트