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.

Sign up for GitHub

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

Jump to bottom

잘못된 도메인 정보 바로잡기 #36

Closed
2 tasks done
dpwns523 opened this issue Aug 16, 2022 · 0 comments · Fixed by #38
Closed
2 tasks done

잘못된 도메인 정보 바로잡기 #36

dpwns523 opened this issue Aug 16, 2022 · 0 comments · Fixed by #38
Assignees
@dpwns523
Labels
enhancement New feature or request
Milestone
게시판 프로젝트 만들기

Comments

@dpwns523
Copy link
Owner

dpwns523 commented Aug 16, 2022 •
edited
Loading

#31 기능을 하다가 도메인 코드 설계가 일부 잘못되어 있는 것을 발견
UserAccount 회원 계정의 userId는 회원 id 이므로 유니크해야 하는데, 해당 속성이 빠져있었다.
erd 문서에는 email의 유니크 키가 표현되지 않음

  • user_id에 유니크 키 추가
  • email erd 업데이트
@dpwns523 dpwns523 self-assigned this Aug 16, 2022
dpwns523 added a commit that referenced this issue Aug 16, 2022
@dpwns523
#36 - erd 업데이트: FK, UK 설정
7a620ed
dpwns523 added a commit that referenced this issue Aug 16, 2022
@dpwns523
#36 - 도메인 업데이트: UserAccount 회원 ID에 유니크 키 추가
d6d9a56 
회원 id로 로그인하고 유저 식별하므로, 당연히 uk 여야 한다.
이 부분이 설계에서 반영되지 않았던 것을 발견
테스트는 uk 적용으로 기존 `data.sql`의 테스트 데이터와
중복이 발생하므로 `userId` 이름 수정
추가적으로 데이터의 수가 123개인데 124여서 변경함
@dpwns523 dpwns523 mentioned this issue Aug 16, 2022
Merged
dpwns523 added a commit that referenced this issue Aug 16, 2022
@dpwns523
Merge pull request #38 from dpwns523/feature/#36-domain-update
7faa223 
잘못된 도메인 정보 바로잡기
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dpwns523 dpwns523

Labels
enhancement New feature or request
Projects
None yet
Milestone
게시판 프로젝트 만들기
Development

Successfully merging a pull request may close this issue.

잘못된 도메인 정보 바로잡기 dpwns523/project-board
1 participant
@dpwns523