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

Member, GroupMember, MemberGroup 엔티티 재설계 #1559

Closed
wonyongChoi05 opened this issue Sep 27, 2023 · 0 comments · Fixed by #1605
Closed

Member, GroupMember, MemberGroup 엔티티 재설계 #1559

wonyongChoi05 opened this issue Sep 27, 2023 · 0 comments · Fixed by #1605
Assignees
Labels
BE 고도화 고도화 스쿼드 관련 이슈

Comments

@wonyongChoi05
Copy link
Member

wonyongChoi05 commented Sep 27, 2023

Member


평범한 멤버 도메인이다. Role이 CREW인 Member만 MemberGroup이 필요하지 않을까 생각했는데 코치까지 크루로 지정되어 있다.

스크린샷 2023-09-27 16 16 53

그래서 Role이라는 권한에서 CREW 밖에 사용하지 않으니, Role의 존재가 필요하지 않다고 생각했는데, 아티클 팀에서 UNVALIDATED 권한을 GUEST로 변경한 뒤 작업 예정

GroupMember


Member 테이블과 MemberGroup 테이블의 매핑 테이블 다대다 관계를 위해 매핑테이블을 만든 것 같은데, 이거 건들면 대공사여서 점진적으로 개선하기

MemberGroup


  • name : 기수 + 포지션(ex: 3기 백엔드 - 오타아님)
  • description : 설명인 것 같은데, 별 의미 없음(ex: 3기 백엔드 - 오타아님)

현재 name과 description이 똑같은 상황. description이 nullable한 레코드도 존재한다. 어드민 페이지가 있는것도 아니고(있긴한데 사용X), 코치 권한을 가진 유저가 특별한 기능을 가진것도 아니기 때문에 MemberGroup이 Member 내로 합쳐져도 무방할 것 같음. BUT 곧 코치, 어드민에게 권한이 주어질 예정이고, 게스트에게 제공하는 아티클 기능이 간단하지 않기 때문에 분리하지 않기로 함


결론

  • name 컬럼 이름 변경
    • 기존 데이터 변경
  • description 컬럼 이름 변경
    • 기존 데이터 변경
  • MemberGroup GroupMember 리팩터링
    • Member는 아티클 팀에서 리팩터링(UNVALIDATED -> GUEST)
@wonyongChoi05 wonyongChoi05 added BE 고도화 고도화 스쿼드 관련 이슈 labels Sep 27, 2023
@wonyongChoi05 wonyongChoi05 self-assigned this Sep 27, 2023
@wonyongChoi05 wonyongChoi05 linked a pull request Oct 13, 2023 that will close this issue
wonyongChoi05 added a commit that referenced this issue Oct 18, 2023
Member, GroupMember, MemberGroup 엔티티 재설계
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 고도화 고도화 스쿼드 관련 이슈
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant