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

v2.1.0 #402

Merged
merged 9 commits into from
Sep 22, 2024
Merged

v2.1.0 #402

merged 9 commits into from
Sep 22, 2024

Conversation

mikekks
Copy link
Member

@mikekks mikekks commented Sep 22, 2024

👩‍💻 Contents

  • v2.1.0 배포

📝 Review Note

📣 Related Issue

✅ 점검사항

  • docker-compose.yml 파일에 마이그레이션 한 API의 포워딩을 변경해줬나요?
  • Spring Secret 값을 수정하거나 추가했다면 Github Secret에서 수정을 해줬나요?
  • Nestjs Secret 값을 수정하거나 추가했다면 Docker-Compose.yml 파일 및 인스턴스 내부의 .env 파일을 수정했나요?

hoonyworld and others added 9 commits September 21, 2024 20:24
* chore(PostDetailBaseDto): 불필요한 import문 삭제

* feat(PostDetailResponseDto): 차단된 유저의 게시물인지 여부를 나타내는 필드 추가

* feat(UserV2Service): id를 통해 User 객체를 가져오는 메서드 선언

* feat(UserV2ServiceImpl): id를 통해 User 객체를 가져오는 메서드 구현

* refactor: 매개변수로 userId가 아닌 user 객체를 받도록 변경

* feat(PostSearchRepositoryImpl): 차단된 유저의 게시물 여부를 확인하는 서브쿼리 추가

* test(PostRepositoryTest): 테스트 임시 비활성화

* feat(PostDetailWithBlockStatusResponseDto): 게시글 객체 + 차단된 유저의 게시물 여부를 나타내는 응답 DTO 생성

* refactor(PostV2GetPostsResponseDto): DTO 객체를 PostDetailResponseDto에서 PostDetailWithBlockStatusResponseDto로 수정

* chore(PostDetailBaseDto): description 상세화

* refactor(PostDetailResponseDto): isBlockedPost 필드 삭제 및 description 상세화

* refactor(PostSearchRepositoryImpl): 차단된 유저의 게시물 여부 확인 서브쿼리 삭제

* feat(MemberBlockSearchRepository): 특정 사용자가 다른 사용자를 차단했는지 여부를 확인하는 메서드 선언

* refactor(MemberBlockRepository): 커스텀 쿼리를 메서드를 구현하기 위한 MemberBlockSearchRepository상속

* feat(MemberBlockRepositoryImpl): 회원 차단 여부 확인을 위한 쿼리 메서드 추가

* feat(MemberBlockService): 회원 차단 여부 확인을 위한 MemberBlockService 인터페이스 정의

* feat(MemberBlockServiceImpl): 회원 차단 여부 확인메서드 구현

* refactor(PostV2ServiceImpl):회원 차단 여부를 DTO로 넘겨주도록 변경

* feat(JpaAuditingConfig): multi-datasource 환경을 위한 JPAQueryFactory 설정 추가

- primaryEntityManager와 playgroundEntityManager를 위한 JPAQueryFactory 빈 등록
- QueryDSL을 사용하여 다양한 데이터 소스에 대한 쿼리 수행 가능

* fix(MemberBlockRepositoryImpl):  playgroundQueryFactory를 주입하여 다중 데이터소스 지원

* refactor(MemberBlockSearchRepository): 차단자가 특정 차단된 사용자 목록에 대해 각각의 차단 상태를 확인하는 메서드로 변경

* refactor(MemberBlockRepositoryImpl): 단일 쿼리로 차단자가 특정 차단된 사용자 목록에 대해 각각의 차단 상태를 확인하는 메서드 구현

* refactor(MemberBlockService): 차단자가 특정 차단된 사용자 목록에 대해 각각의 차단 상태 정보를 얻을 수 있는 메서드 선언

* refactor(MemberBlockServiceImpl): 차단자가 특정 차단된 사용자 목록에 대해 각각의 차단 상태 정보를 얻을 수 있는 메서드 구현

* rename(MemberBlockSearchRepositoryImpl): MemberBlockRepositoryImpl에서 MemberBlockSearchRepositoryImpl로 클래스 이름 변경

* refactor: 매개변수 이름 변경

* refactor(PostV2ServiceImpl): 조회된 게시물 작성자의 orgId 리스트를 수집하여 차단 여부를 확인하는 로직 최적화

* rename(PostDetailWithBlockStatusResponseDto): 응답 필드명 변경

* chore: naver 코드 포맷터 적

* refactor: 매개변수를 userId로 받도록 변경

* refactor: osiv 관련 코드리뷰 반

* test: test yml 수정

* chore(config): test 설정 추가, 테스트 프로필일 경우 schema.sql 실행

* test(config): 테스트컨테이너 사용하도록 코드 전면 수정

* chore(MemberBlock): [중요] querydsl -> spring data jpa 변경으로 인한 임시 수정

* refactor: 역할에 맞게 서비스 레이어에서 Map으로 변환하도록 리팩토링

* test(PostRepositoryTest): 주석 해제

* delete: querydsl 대신 spring data jpa를 사용하기 위해 삭제

- spring data jpa의 메서드 네이밍이 복잡하지 않으며 성능차이도 비슷하기 때문에 코드를 줄이는 방향으로 결정

* feat(MemberBlockRepository): 차단자의 orgId로 차단된 모든 유저의 정보를 조회하는 메서드 추가

* refactor(MemberBlockServiceImpl): spring data jpa를 이용해 차단된 유저의 정보를 Map으로 반환하도록 변경

* chore(application-test.yml): 테스트 컨테이너 jdbc-url 변경

---------

Co-authored-by: mikekks <[email protected]>
* chore(PostDetailBaseDto): 불필요한 import문 삭제

* feat(PostDetailResponseDto): 차단된 유저의 게시물인지 여부를 나타내는 필드 추가

* feat(UserV2Service): id를 통해 User 객체를 가져오는 메서드 선언

* feat(UserV2ServiceImpl): id를 통해 User 객체를 가져오는 메서드 구현

* refactor: 매개변수로 userId가 아닌 user 객체를 받도록 변경

* feat(PostSearchRepositoryImpl): 차단된 유저의 게시물 여부를 확인하는 서브쿼리 추가

* test(PostRepositoryTest): 테스트 임시 비활성화

* feat(PostDetailWithBlockStatusResponseDto): 게시글 객체 + 차단된 유저의 게시물 여부를 나타내는 응답 DTO 생성

* refactor(PostV2GetPostsResponseDto): DTO 객체를 PostDetailResponseDto에서 PostDetailWithBlockStatusResponseDto로 수정

* chore(PostDetailBaseDto): description 상세화

* refactor(PostDetailResponseDto): isBlockedPost 필드 삭제 및 description 상세화

* refactor(PostSearchRepositoryImpl): 차단된 유저의 게시물 여부 확인 서브쿼리 삭제

* feat(MemberBlockSearchRepository): 특정 사용자가 다른 사용자를 차단했는지 여부를 확인하는 메서드 선언

* refactor(MemberBlockRepository): 커스텀 쿼리를 메서드를 구현하기 위한 MemberBlockSearchRepository상속

* feat(MemberBlockRepositoryImpl): 회원 차단 여부 확인을 위한 쿼리 메서드 추가

* feat(MemberBlockService): 회원 차단 여부 확인을 위한 MemberBlockService 인터페이스 정의

* feat(MemberBlockServiceImpl): 회원 차단 여부 확인메서드 구현

* refactor(PostV2ServiceImpl):회원 차단 여부를 DTO로 넘겨주도록 변경

* feat(JpaAuditingConfig): multi-datasource 환경을 위한 JPAQueryFactory 설정 추가

- primaryEntityManager와 playgroundEntityManager를 위한 JPAQueryFactory 빈 등록
- QueryDSL을 사용하여 다양한 데이터 소스에 대한 쿼리 수행 가능

* fix(MemberBlockRepositoryImpl):  playgroundQueryFactory를 주입하여 다중 데이터소스 지원

* refactor(MemberBlockSearchRepository): 차단자가 특정 차단된 사용자 목록에 대해 각각의 차단 상태를 확인하는 메서드로 변경

* refactor(MemberBlockRepositoryImpl): 단일 쿼리로 차단자가 특정 차단된 사용자 목록에 대해 각각의 차단 상태를 확인하는 메서드 구현

* refactor(MemberBlockService): 차단자가 특정 차단된 사용자 목록에 대해 각각의 차단 상태 정보를 얻을 수 있는 메서드 선언

* refactor(MemberBlockServiceImpl): 차단자가 특정 차단된 사용자 목록에 대해 각각의 차단 상태 정보를 얻을 수 있는 메서드 구현

* rename(MemberBlockSearchRepositoryImpl): MemberBlockRepositoryImpl에서 MemberBlockSearchRepositoryImpl로 클래스 이름 변경

* refactor: 매개변수 이름 변경

* refactor(PostV2ServiceImpl): 조회된 게시물 작성자의 orgId 리스트를 수집하여 차단 여부를 확인하는 로직 최적화

* rename(PostDetailWithBlockStatusResponseDto): 응답 필드명 변경

* chore: naver 코드 포맷터 적

* refactor: 매개변수를 userId로 받도록 변경

* refactor: osiv 관련 코드리뷰 반

* test: test yml 수정

* chore(config): test 설정 추가, 테스트 프로필일 경우 schema.sql 실행

* test(config): 테스트컨테이너 사용하도록 코드 전면 수정

* chore(MemberBlock): [중요] querydsl -> spring data jpa 변경으로 인한 임시 수정

* refactor: 역할에 맞게 서비스 레이어에서 Map으로 변환하도록 리팩토링

* test(PostRepositoryTest): 주석 해제

* delete: querydsl 대신 spring data jpa를 사용하기 위해 삭제

- spring data jpa의 메서드 네이밍이 복잡하지 않으며 성능차이도 비슷하기 때문에 코드를 줄이는 방향으로 결정

* feat(MemberBlockRepository): 차단자의 orgId로 차단된 모든 유저의 정보를 조회하는 메서드 추가

* refactor(MemberBlockServiceImpl): spring data jpa를 이용해 차단된 유저의 정보를 Map으로 반환하도록 변경

* chore(application-test.yml): 테스트 컨테이너 jdbc-url 변경

* refactor(PostDetailResponseDto): isBlockedPost 필드 추가

* refactor(PostSearchRepositoryImpl): 정적팩토리 메서드 호출 시 isBlockedPost 필드를 임시로 null로 초기화

* refactor(PostV2GetPostsResponseDto): PostDetailResponseDto 객체 리스트 필드로 변경

* refactor(PostV2ServiceImpl): PostDetailResponseDto 객체로 변경 및 isBlockedPost를 true/false로 초기화하도록 변경

* refactor(PostSearchRepositoryImpl): isBlockedPost를 false로 초기화하도록 변경

* refactor(PostDetailResponseDto): isBlockedPost 자료형을 널값을 허용하지 않는 boolean으로 변경

* delete(PostDetailWithBlockStatusResponseDto): PostDetailWithBlockStatusResponseDto 삭제

* refactor(PostV2ServiceImpl): DTO 반환 로직 private 메서드로 분리

---------

Co-authored-by: mikekks <[email protected]>
* del: 임시 코드 삭제

* chore(comment): 차단 여부 필드 추가

* feat(comment): 차단 유저 Map 반환 메서드 구현

* feat(comment): 차단 댓글 필터링 구현

* fix(comment): 삭제된 댓글에 대한 처리
* refactor(const): final 클래스로 변경

* chore(const): 현재 기수 35기로 수정

* chore(const): 생성자 -> lombok 으로 수정
* refactor(CommentDto): Lombok getter에서 isBlockedComment를 생략 후 직접 getter 구현

* refactor(ReplyDto): Lombok getter에서 isBlockedComment를 생략 후 직접 getter 구현
@mikekks mikekks added 🚀 deployment 정기 배포와 관련된 라벨 2️⃣ minor labels Sep 22, 2024
@mikekks mikekks requested a review from hoonyworld September 22, 2024 17:17
@mikekks mikekks self-assigned this Sep 22, 2024
Copy link

height bot commented Sep 22, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@mikekks mikekks merged commit 7369ea5 into main Sep 22, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2️⃣ minor 🚀 deployment 정기 배포와 관련된 라벨 size/XXL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants