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

[공통] 전반적인 프로젝트 아키텍처 변경 #305

Merged
merged 102 commits into from
Mar 29, 2024
Merged

[공통] 전반적인 프로젝트 아키텍처 변경 #305

merged 102 commits into from
Mar 29, 2024

Conversation

kylekim2123
Copy link
Member

@kylekim2123 kylekim2123 commented Mar 29, 2024 •
edited
Loading

👨‍💻 작업 사항

📑 PR 개요

  • 프로젝트 아키텍처를 변경하고 그에따라 코드를 수정한다.

✅ 작업 목록

  • Member, Crew, Game, Chat, Address 도메인에 구현 계층 추가하고, 비즈니스 로직을 이관
  • Member, Crew, Game, Chat, Address 도메인에서 엔티티와 도메인 객체를 분리
  • 리팩토링 이후 모든 API에 대한 통합 테스트 진행

kylekim2123 and others added 30 commits February 2, 2024 13:44
@kylekim2123
refactor: Crew 도메인에서 JPA 양방향 연관관계 제거
401ff03
@kylekim2123
test: 크루 도메인 양방향 연관관계 제거로 인한, 테스트 코드 에러 해결
2452fe5
@Hchanghyeon
refactor: MemberService 분리 및 Repository default 메서드 추가
1d22e43
@Hchanghyeon
refactor: 양방향 연관 관계 분리
5acc46e
@Hchanghyeon
refactor: Member 양방향 연관 관계 제거 후 서비스 로직 리팩토링
fc77682
@Hchanghyeon
refactor: Member와 관련된 Embeddable 클래스 삭제 및 Member 연관 관계 주석 제거
0ffd58c
@Hchanghyeon
refactor: Member 연관 관계 제거 후 발생되는 에러 조치
9e8d445
@Hchanghyeon
refactor: Member 연관 관계 제거 후 발생되는 테스트 에러 조치
3cf5780
@Hchanghyeon
refactor: rebase 후 에러 조치
4ef9ba6
@charlesuu
refactor: GameService 분리
d5c9a6f
@charlesuu
refactor: 양방향 연관 관계 분리
f5fcb03
@kylekim2123
refactor: 프로젝트 전체 코드 리포매팅
aa26737
@kylekim2123
refactor: Chat 도메인 양방향 연관관계 제거
6efa8aa
@Hchanghyeon @charlesuu
refactor: Member 도메인에서 Address 연관관계 삭제, 관련 에러 조치
a0d3776
@Hchanghyeon @charlesuu
refactor: 불필요한 클래스 삭제
45bb117
@Hchanghyeon @charlesuu
refactor: MapService 수정
9f5ba68
@kylekim2123
fix: MainAddressId -> MainAddress로 네이밍 변경하여 컴파일 에러 해결
2e23703
@Hchanghyeon
fix: 포지션 중복 확인 예외 처리 추가
2c02907
@Hchanghyeon
style: 컨벤션 수정
17de837
@charlesuu
refactor: addressReader에 readMainAddressByGame 메서드 추가, readMainAddres…
cb0ece5 
…s메서드명 readMainAddressByMember로 변경
@charlesuu
refactor: Game의 Address에 대한 단방향 연관관계 제거
e6ebdb1
@charlesuu
refactor: game - address 연관관계 제거로 인한 toDTO, fromDTO 시그니처 변경
fdb4ea7
@kylekim2123
refactor: 불필요한 클래스, 메서드, 주석 삭제
0548aa9
@kylekim2123
refactor: Crew 도메인 Address 단방향 연관관계 제거
07441ac
@charlesuu
refactor: Game 단방향 연관관계 제거로 인한 queryDSL 조회 방식 변경
c676936
@Hchanghyeon
refactor: Member, Crew 도메인 연관관계 모두 제거
628c246
@Hchanghyeon
refactor: Member, Crew 도메인 연관관계 제거 추가 작업
661a659
@kylekim2123
refactor: Chat 도메인 단방향 연관관계 삭제
1e68c4d
@charlesuu
refactor: Game 도메인 연관관계 제거
05d20a6
@charlesuu
refactor: GameMember 연관관계 제거
626c2b4
kylekim2123 and others added 18 commits March 9, 2024 14:14
@kylekim2123
refactor: 사용자의 참여 확정 게스트 모집글 목록 조회 기능에 대한 구현계층 적용
bfd5f46
@kylekim2123
refactor: 사용자가 만든 게스트 모집글 목록 조회 기능에 대한 구현계층 적용
051e72d
@kylekim2123
refactor: Crew, Game 엔티티 변경에 의한 알람 로직 수정
e03b8c0
@kylekim2123
refactor: 사용자의 크루, 게스트 관련 조회 로직에 대해 구현계층 적용 및 각 도메인으로 로직 이관
51047f8
@kylekim2123
refactor: 주소 도메인 구현계층 적용
0e14593
@kylekim2123
refactor: 주소 도메인의 AddressDepth1, AddressDepth2 엔티티 객체에 대해 repository의…
b9589e8 
… entity 폴더로 이관 및 변수명 변경
@kylekim2123
fix: MemberRepository의 updateMannerScore 메서드에 대한 로직이 비어있어서 JPQL로 새로 작성
4bc0511
@charlesuu
refactor: member_position이 여러 개 생성될 때 bulk insert쿼리가 나가도록 개선
3ec1e6e
@charlesuu
refactor: game_position들이 생성될 때 bulk insert쿼리가 나가도록 개선
4e51da3
@kylekim2123
refactor: 각 도메인의 entity 클래스에 Table 이름 설정
be64834
@Hchanghyeon
refactor: Entity에 주소 컬럼명 명시
67e030e
@kylekim2123
refactor: update 쿼리에 @Modifying 애너테이션 추가
3864c01
@kylekim2123
fix: 특정 채팅방의 모든 메시지 조회 기능에서 다른 채팅방의 메시지도 가져오는 문제 해결
85bbd8d
@Hchanghyeon
refactor: Game 도메인 관련 로직 수정
51f7d4a
@Hchanghyeon
refactor: JPQL Modifying 적용
5a02baa
@Hchanghyeon
refactor: caffeineCache keyName변경
bae142b
@Hchanghyeon
refactor: http 파일 내용 수정 및 추가
3a05546
@Hchanghyeon
refactor: 메서드 매개변수 순서 변경
d9e9883
@kylekim2123 kylekim2123 added 코드 스타일 [style] 변수/클래스/메서드 명 변경, 들여쓰기, 개행 등 각종 포맷팅 스타일을 변경한다. 버그 [fix] 에러를 해결하거나, 잘못된 결과를 수정한다. 리팩토링 [refactor] 실행 결과는 동일하고, 코드 구조만 변경한다. 기타 [chore] 패키지 구조 변경 및 삭제, 이외의 기타 작업을 수행한다. labels Mar 29, 2024
@kylekim2123 kylekim2123 merged commit ee50a95 into dev Mar 29, 2024
1 check failed
@kylekim2123 kylekim2123 deleted the refactoring branch March 29, 2024 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@charlesuu charlesuu charlesuu approved these changes

@Hchanghyeon Hchanghyeon Awaiting requested review from Hchanghyeon

Assignees

@kylekim2123 kylekim2123

@charlesuu charlesuu

@Hchanghyeon Hchanghyeon

Labels
기타 [chore] 패키지 구조 변경 및 삭제, 이외의 기타 작업을 수행한다. 리팩토링 [refactor] 실행 결과는 동일하고, 코드 구조만 변경한다. 버그 [fix] 에러를 해결하거나, 잘못된 결과를 수정한다. 코드 스타일 [style] 변수/클래스/메서드 명 변경, 들여쓰기, 개행 등 각종 포맷팅 스타일을 변경한다.
Projects
Pickple-back
Status: 코드 리뷰
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kylekim2123 @charlesuu @Hchanghyeon