Skip to content

연애고사 이미지 업로드 기능 추가#443

Merged
stemmmm merged 99 commits into
mainfrom
feat/442
May 2, 2026
Merged

연애고사 이미지 업로드 기능 추가#443
stemmmm merged 99 commits into
mainfrom
feat/442

Conversation

@stemmmm
Copy link
Copy Markdown
Contributor

@stemmmm stemmmm commented May 2, 2026

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능

    • 자기소개에 이미지 업로드 기능 추가
    • 이미지 업로드를 위한 사전 서명된 URL 생성 엔드포인트 추가
  • 개선 사항

    • S3 업로드 인프라 구조 정리 및 최적화
    • CI/CD 워크플로우 트리거 조건 개선
    • 자기소개 관련 데이터베이스 스키마 업데이트
  • 테스트

    • 신규 이미지 기능 관련 테스트 커버리지 확대

참고 자료

노트

stemmmm and others added 30 commits December 12, 2025 12:23
- 인터뷰 질문, 미션, 데이팅 시험 관련 기본 데이터 마이그레이션 추가
- Flyway 설정 로직에 비활성화 처리 조건 추가
- Hibernate DDL 설정 방식 변경 및 스키마 생성 스크립트 설정 추가

#377
# Conflicts:
#	src/main/java/deepple/deepple/common/config/FlywayConfig.java
- k6 테스트 설정 파일 및 API 테스트 스크립트 추가
- 테스트 환경 변수 및 부하 프로파일 설정
- README에 k6 관련 문서 및 실행 방법 추가

#394
- 성능 테스트용 SQL 데이터 생성 스크립트 추가
- 회원, 소개, 좋아요, 매칭, 알림 데이터 포함
- 데이터 검증 및 README 파일 추가

#394
- FCM 제외 테스트 로직 `sendWithoutPush` 추가
- k6 부하 테스트 스크립트 구현
- 테스트용 SQL 데이터 초기화/생성 로직 추가

#394
* [Refac] : When Match Request or Response, Don't Check Primary Contatct Type

* [fix] : Change BaseURL

* [Fix] : Using Enum
* [Refac] : When Match Request or Response, Don't Check Primary Contatct Type

* [fix] : Change BaseURL

* [Fix] : Using Enum

* [Refac] Change BizgoRequestBody
* [Refac] : When Match Request or Response, Don't Check Primary Contatct Type

* [fix] : Change BaseURL

* [Fix] : Using Enum

* [Refac] Change BizgoRequestBody

* [Fix] : Use Record for serialization
* [Refac] : When Match Request or Response, Don't Check Primary Contatct Type

* [fix] : Change BaseURL

* [Fix] : Using Enum

* [Refac] Change BizgoRequestBody

* [Fix] : Use Record for serialization

* [Refac] : Add Logging for checking.
- 데이터 접근 레이어 분리 (Reader/Writer) 및 저장 로직 개선
- FCM 예외 처리 로깅 추가
- 알림 템플릿 정보 전송 방식 변경
- k6 부하 테스트 시 최대 VU 증가
- notification_preferences 테이블에 인덱스 추가
- 테스트 코드 리팩토링 및 명확한 실패 상태 로깅 적용

#394
- CircuitBreaker 상태 전환, 실패율 초과, 느린응답률 초과 이벤트에 대한 로그 등록
- 이벤트 리스너 등록 로직 추가 및 @slf4j 어노테이션 적용
- 알림 테이블에 복합 인덱스 추가 (receiverId, deletedAt, id)
- 데이터베이스 및 엔티티 매핑 수정

#394
- Bizgo API 응답에서 에러 상태 코드 및 본문 로깅 추가
- 예외 발생 시 디버깅 용이하도록 개선

#409
stemmmm and others added 26 commits February 24, 2026 23:26
- 리프레시 토큰 처리 및 재발급 관련 코드 제거
- `TokenFilter`의 불필요한 의존성 삭제
- 인증 로직 단순화 및 불필요한 메서드 제거
- Google Play 클라이언트 및 API 검증 로직 추가
- Google Play 서비스 계정 인증 구성
- Google Play 결제 영수증 검증 및 이벤트 처리 로직 구현
- Google Play 관련 예외, DTO, 설정, 테스트 코드 추가

#438
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- 셀프 소개에 이미지 URL 필드 도입
- Pre-signed URL 생성 API 구현

#442
@stemmmm stemmmm self-assigned this May 2, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 2, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f6a0b126-83ee-4cbf-b0b1-c3554cc9f72f

📥 Commits

Reviewing files that changed from the base of the PR and between e53494e and 62fee2e.

📒 Files selected for processing (23)
  • .github/workflows/cd-dev.yml
  • .github/workflows/ci.yml
  • src/main/java/deepple/deepple/common/infra/s3/S3Uploader.java
  • src/main/java/deepple/deepple/common/infra/s3/dto/PresignedUrlResponse.java
  • src/main/java/deepple/deepple/common/infra/s3/exception/S3AmazonException.java
  • src/main/java/deepple/deepple/common/infra/s3/exception/S3ClientException.java
  • src/main/java/deepple/deepple/community/command/application/selfintroduction/SelfIntroductionService.java
  • src/main/java/deepple/deepple/community/command/domain/selfintroduction/SelfIntroduction.java
  • src/main/java/deepple/deepple/community/presentation/selfintroduction/SelfIntroductionController.java
  • src/main/java/deepple/deepple/community/presentation/selfintroduction/dto/SelfIntroductionWriteRequest.java
  • src/main/java/deepple/deepple/community/query/selfintroduction/AdminSelfIntroductionQueryRepository.java
  • src/main/java/deepple/deepple/community/query/selfintroduction/SelfIntroductionQueryRepository.java
  • src/main/java/deepple/deepple/community/query/selfintroduction/view/AdminSelfIntroductionView.java
  • src/main/java/deepple/deepple/community/query/selfintroduction/view/SelfIntroductionSummaryView.java
  • src/main/java/deepple/deepple/community/query/selfintroduction/view/SelfIntroductionView.java
  • src/main/java/deepple/deepple/member/command/application/profileImage/ProfileImageService.java
  • src/main/java/deepple/deepple/member/presentation/profileimage/ProfileImageController.java
  • src/main/resources/db/migration/V15__add_image_url_to_self_introductions.sql
  • src/test/java/deepple/deepple/community/command/application/selfintroduction/SelfIntroductionServiceTest.java
  • src/test/java/deepple/deepple/community/command/domain/selfintroduction/SelfIntroductionTest.java
  • src/test/java/deepple/deepple/community/query/AdminSelfIntroductionQueryRepositoryTest.java
  • src/test/java/deepple/deepple/community/query/SelfIntroductionQueryRepositoryTest.java
  • src/test/java/deepple/deepple/member/command/application/profileimage/ProfileImageServiceTest.java

Walkthrough

이 PR은 세 가지 주요 변경 사항을 포함합니다. 첫째, CI/CD 워크플로우가 개선되어 개발 배포는 workflow_run 이벤트 대신 develop 브랜치에 대한 직접 push 이벤트로 트리거되고, 테스트 작업이 Gradle wrapper 검증 및 JDK 21 설정을 포함하도록 확장되었습니다. 둘째, S3 관련 클래스(S3Uploader, PresignedUrlResponse, 예외 클래스들)가 멤버 프로필 이미지 패키지에서 공통 인프라 패키지로 이동되었습니다. 셋째, 자기소개 기능이 이미지 URL 지원을 추가하여 데이터베이스 스키마 업데이트, 엔티티 및 서비스 확장, 새로운 presigned URL 엔드포인트 추가, 그리고 관련 뷰 및 테스트 업데이트를 포함합니다.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~70 minutes

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/442

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
Review rate limit: 0/1 reviews remaining, refill in 60 minutes.

Comment @coderabbitai help to get the list of available commands and usage tips.

@stemmmm stemmmm merged commit 7361e33 into main May 2, 2026
2 of 3 checks passed
@stemmmm stemmmm deployed to development May 2, 2026 04:28 — with GitHub Actions Active
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants