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

Member Entity 추가 이후 DIR 도메인의 연관관계 추가 및 로직 수정 #19

Merged
merged 2 commits into from
Aug 9, 2022
Merged

Member Entity 추가 이후 DIR 도메인의 연관관계 추가 및 로직 수정 #19

merged 2 commits into from
Aug 9, 2022

Conversation

PPakSang
Copy link
Contributor

@PPakSang PPakSang commented Aug 9, 2022

PR 요약

  1. InDIR - Member 연관관계 추가 및 Builder 패턴 개선
  2. Member 추가에 따른 DIRRepository 수정

변경 사항

  1. InDIR Entity 의 Builder 패턴 개선 (기존 setter 를 통한 path, name 설정에서 한번에 builder 패턴으로 설정 가능하도록 수정)
  2. member_id + path + name 으로 Entity 를 조회하도록 로직 전면 수정

참고 사항

PPakSang added 2 commits August 9, 2022 21:56
@PPakSang
Refactor: InDIR - Member 연관관계 추가 및 Builder 패턴 개선
019d75f 
InDIR Entity 와 Member Entity 와 연관관계를 추가합니다.

기존 상속관계 부모 필드를 setter 를 통해서 주입하는 방식을 사용하였는데, 해당 방식을 위한 setter 를 따로 두는 것이 Builder 로 온전한 객체를 생성하지 못하는 비효율적인 코드라고 판단하여 이를 삭제하고, Builder 내부에서 생성가능한 로직으로 수정했습니다
@PPakSang
Refactor: Member 추가에 따른 DIRRepository 수정
7f2266b 
member 가 추가됨에 따라 path+name 이 더이상 식별자의 역할을 할 수 없기 때문에 member_id 를 포함한 조회 로직으로 코드를 전면적으로 수정합니다.
@PPakSang PPakSang added the documentation Improvements or additions to documentation label Aug 9, 2022
@PPakSang PPakSang requested review from hw0603, HyeW and dkdud9261 August 9, 2022 13:03
@PPakSang PPakSang self-assigned this Aug 9, 2022
@PPakSang PPakSang linked an issue Aug 9, 2022 that may be closed by this pull request
연관관계 매핑 #12
Open
@PPakSang PPakSang merged commit 09c78f5 into develop Aug 9, 2022
@PPakSang PPakSang linked an issue Aug 10, 2022 that may be closed by this pull request
Refactor: DIRRepository 조회 방식 수정 #18
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hw0603 hw0603 Awaiting requested review from hw0603

@HyeW HyeW Awaiting requested review from HyeW

@dkdud9261 dkdud9261 Awaiting requested review from dkdud9261

Assignees

@PPakSang PPakSang

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Refactor: DIRRepository 조회 방식 수정 연관관계 매핑
1 participant
@PPakSang