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

채팅방 목록 조회 시 쿼리 변경 #760

Closed
wants to merge 10 commits into from
Closed

채팅방 목록 조회 시 쿼리 변경 #760

wants to merge 10 commits into from

Conversation

kwonyj1022
Copy link
Collaborator

📄 작업 내용 요약

지토가 suggestion 올려주신 내용이고 추가적인 수정 사항은 없습니다.

🙋🏻 리뷰 시 주의 깊게 확인해야 하는 코드

📎 Issue 번호

JJ503 and others added 10 commits November 1, 2023 16:03
@JJ503
docs: 문서 최신화 (#721)
2f6c9a0
@kwonyj1022
remove: #713 사용하지 않는 초기데이터 세팅 제거 (#716)
96decd3
@JJ503 @kwonyj1022
ci: #728 flyway ci 검증 추가 (#730)
f868d3a 
* chore: flyway 플러그인 추가

* ci: dev flyway ci 검증 설정 추가

* ci: flyway 검증 설정 파일 수정

* ci: flyway 검증 설정 수정

* ci: flyway ci 설정 수정

* ci: flyway ci 설정 수정

* ci: flyway ci 검증 스크립트 수정

* ci: flyway ci 스크립트 수정

* ci: flyway ci 스크립트 수정

* ci: flyway ci 검증 스크립트 수정

* �ci: flyway ci 스크립트 수정

* ci: flyway ci 검증 스크립트 수정

* ci: flyway ci 검증 스크립트 수정

* �ci: flyway ci 검증 스크립트 추가

* ci: flyway ci 검증 스크립트 수정

* ci: flyway ci 검증 스크립트 수정

* ci: db url 수정

* db url 수정

* ci: mysql 실행 시 sleep 추가

* ci: 잘못된 mysql url 수정

* ci: mysql 연결이 잘 됐는지 확인하는 로직 추가

* ci: mysql service 추가

* ci: url에 포트 변경

* ci: flyway ci 검증 스크립트 수정

* ci: docker를 실행하는 로직 추가

* ci: flyway ci 스크립트 수정

* ci: flyway ci 스크립트 수정

* ci: flyway 검증을 위한 docker-compose.yml 추가

* ci: docker 실행 명령어 추가

* �ci: docker-compose 경로 지정 변경

* ci: flyway ci 검증 스크립트 수정

* ci: flyway ci 검증 스크립트 수정

* ci: flyway ci 검증 스크립트 수정

* ci: flyway ci 검증 스크립트 수정

* ci: docker compose start 확인

* ci: $DATABASE_URL를 가져오는 지 확인

* ci: $DATABASE_URL 출력 위치 수정

* ci: flyway 검증 로직 추가

* ci: docker 검증 로직 추가

* ci: 포트 수정

* ci: 포트 수정

* ci: talnet 제거

* ci: url 수정

* chore: flyway 의존성 추가

* ci: 경로 수정

* chore: build.gradle 스크립트 수정

* ci: migration 폴더에 수정 사항이 있어야지만 수행하도록 수정

* test: flyway ci 스크립트 수행 테스트

* ci: 경로 수정

* test: flyway ci 스크립트 수행 테스트

* ci: 경로 수정

* ci: 파일 명과 경로 수정

* ci: 경로 수정

* ci: 경로 수정

* ci: 경로 수정

* test: flyway ci 제거 후 스크립트 수행 테스트

* ci: 서비스를 통해 mysql 실행

* test: flyway ci 스크립트 수행 테스트

* ci: 포트 번호 수정

* ci: 필요없는 docker compose 파일 제거

* test: 오류가 나는 flyway 스크립트 수행 테스트

* test: 오류가 나는 flyway 스크립트 수행 테스트

* test: 테스트를 위한 스크립트 제거

* ci: pr 슬랙 메시지 중 리뷰어 항목 제거

* ci: 검증 이름 수정

* ci: 메시지에 리뷰 요청에 대한 방법 추가

---------

Co-authored-by: 권예진 <[email protected]>
@swonny
feat: #677 안 읽은 메시지 카운팅 기능 구현 (#709)
4d4c3f7 
* feat: 읽은 메시지 로그 도메인 생성

* feat: 읽은 메시지 로그 저장 레포지토리 생성

* refactor: 컬럼명 변경

* feat: 메시지 조회 시 마지막으로 읽은 메시지 로그 저장

* feat: 채팅방 목록 조회 시 읽지 않은 메시지 개수를 포함하는 기능 추가

* feat: 채팅방 생성 시 참여자에 대한 메시지 로그 생성하는 기능 추가

* feat: 경매 아이디로 채팅방 조회하는 기능 추가

* feat: 채팅방 목록 조회 시 반환값에 안 읽은 메시지 개수 추가

* refactor: 메시지 로그 조회 네이밍 변경

* feat: 어노테이션 추가

* feat: 로그 찾지 못한 경우에 대한 커스텀 예외 추가

* refactor: 로그 생성 로직 이벤트로 분리

* refactor: 불필요한 메서드 삭제

* test: 생략한 테스트 추가

* refactor: 채팅방과 메시지 조회 로그cascade type 지정

* refactor: 불필요한 파라미터 삭제

* feat: flyway 스크립트 작성

* refactor: 불필요한 어노테이션 삭제

* refactor: 개행 추가 및 분기문 스트림으로 대체

* feat: 안 읽은 메시지 개수 컨트롤러 업데이트

* refactor: 불필요한 필드 삭제

* refactor: 불필요한 join이 발생하는 쿼리 삭제

* refactor: 불필요한 import문 삭제

* refactor: 메시지 전송과 읽음 저장 트랜잭션 분리

* refactor: 불필요한 어노테이션 삭제, 자동 정렬

* test: 예외 케이스 테스트 추가

* refactor: 업데이트에 적절한 변수명으로 변경

* refactor: 레포지토리 저장 시 여러 개 저장하는 메서드 생성

* chore: 잘못된 flyway 스크림트 수정
@swonny
fix: #731 읽지 않은 메시지 로그 조회 시 누락된 조건문 추가 (#733)
8cc9218
@kwonyj1022
이미지url에 id 대신 파일 이름을 사용하도록 변경 (#717)
07bad46 
* refactor: 이미지 상대 url 기반으로 url을 계산하는 메서드 파라미터로 이미지 id 대신 이미지 이름을 받도록 변경

* refactor: 이미지 절대 url 기반으로 url을 계산하는 메서드 파라미터로 이미지 id 대신 이미지 이름을 받도록 변경

* refactor: 경매 조회 시 판매자 프로필 이미지 조회로 인한 n+1 개선

* refactor: 입찰 목록 조회 시 입찰자 정보 조회로 인한 n+1 개선

* refactor: 채팅방 관련 조회 시 참여자 프로필 이미지 조회로 인한 n+1 개선

* refactor: qna 관련 조회 시 사용자 프로필 이미지 조회로 인한 n+1 개선

* refactor: 신고 관련 조회 시 사용자 프로필 이미지 조회로 인한 n+1 개선

* refactor: 평가 관련 조회 시 사용자 프로필 이미지 조회로 인한 n+1 개선

* refactor: 사용자 정보 조회 시 사용자 프로필 이미지 조회로 인한 n+1 개선

* refactor: 메시지 알림 시 작성자 프로필 이미지 조회로 인한 n+1 개선

* feat: 이미지 이름이 포함된 url로 조회 시 이미지 파일을 반환하는 기능 추가

* docs: 문서 최신화

* style: 개행 수정 및 누락된 final 추가

* rename: test에 사용되는 fixture 클래스 이름 변경

* ci: 충돌 해결
@kwonyj1022
이미지 S3 업로드 기능 구현 (#707)
a97b42d 
* chore: aws, s3, cloudfront 의존성 추가

* feat: AWS S3 설정 추가

* feat: StoreImageProcessor 상수 추가

* feat: S3 이미지 업로드 기능 추가

* test: S3 이미지 업로드 기능 테스트 추가

* feat: S3 이미지 업로드 기능 사용을 제어할 수 있는 설정 추가

* feat: 테스트 환경에서 S3 이미지 업로드 기능 사용을 제어할 수 있는 설정의 설정값 변경

* remove: 테스트용 AwsConfiguration 삭제

* feat: 프로덕션 환경에서만 동작하는 Profile 어노테이션 추가

* test: 누락된 s3 지역 설정값 추가

* feat: 프로덕션 환경에 s3 관련 yml 설정 추가

* refactor: 누락된 final 추가
@kwonyj1022
fix: #752 aws 관련 configuration import 수정 및 패키지 이동 (#753)
f913455 
* fix: 잘못된 import 수정

* rename: Aws 관련 configuration 파일 패키지 이동
@kwonyj1022
refactor: #756 입찰 시 Optional 사용 로직 변경 (#757)
9a605d8 
* refactor: 입찰 성공 시 알림 전송 관련 Optional 사용 로직 변경

* feat: 경매의 마지막 입찰을 조회하는 기능 추가

* refactor: 입찰 시 입찰하고자 하는 내용이 유효한지 검증하는 로직 변경

* refactor: 사용하지 않는 메서드 제거
@kwonyj1022
refactor: 집계 함수 쿼리 최적화
ad2e8ec
@kwonyj1022 kwonyj1022 added refactor 기존 기능에 변경이 없는 구현 변경 시 backend 백엔드와 관련된 이슈나 PR에 사용 labels Nov 28, 2023
@kwonyj1022 kwonyj1022 added this to the 레벨 5 milestone Nov 28, 2023
@kwonyj1022 kwonyj1022 self-assigned this Nov 28, 2023
@kwonyj1022 kwonyj1022 closed this Nov 28, 2023
@kwonyj1022 kwonyj1022 deleted the refactor/749 branch January 1, 2024 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@apptie apptie Awaiting requested review from apptie

@JJ503 JJ503 Awaiting requested review from JJ503

@swonny swonny Awaiting requested review from swonny

Assignees

@kwonyj1022 kwonyj1022

Labels
backend 백엔드와 관련된 이슈나 PR에 사용 refactor 기존 기능에 변경이 없는 구현 변경 시
Projects
2023-3-ddang
Status: Done
Milestone
레벨 5
Development

Successfully merging this pull request may close these issues.

채팅방 목록 조회 시 쿼리 변경
3 participants
@kwonyj1022 @JJ503 @swonny