Conversation
dbalsk
changed the title
Dimo-2562
requested changes
Mar 7, 2026
| String content, | ||
| MessageType messageType, | ||
| List<ImageInfo> images, | ||
| List<FileInfo> files, |
Contributor
There was a problem hiding this comment.
아 이거 필드명도 바꾸셨군요, 이거 혹시 노션에 반영도 해주실 수 있나요? 아마 그러면 프론트측에서 에러가 날 거 같아서 프론트측에서도 전면적인 수정이 보입니다.
아니면 필드명은 그대로 가는 것도 괜찮을 거 같기도 합니다.
Merged
kanghana1
added a commit
that referenced
this pull request
Mar 23, 2026
* [chore/#528] 서브도메인 명시 (#529) * [chore/#531] Nginx 설정 파일 수정 (#532) * chore: Nginx 프로토콜 설정을 Cloudflare에서 전달받아 설정하도록 변경 * chore: Nginx 웹소켓 타임아웃의 기본값이 1분이므로 60분으로 변경 * chore: Nginx gzip 설정을 통해 json 응답은 압축해서 보내도록 변경 * [Fix/#521] 채팅 이미지/파일 API 통합 및 테스트 코드 작성 (#526) * refact: image -> file로 통합 * refact: S3 -> GCS로 예외 코드 명칭 변경 * refact: 채팅 이미지 업로드/다운로드 코드 제거 * refact: 채팅 이미지 관련 코드 수정 * feat: 채팅 파일 통합테스트 * feat: 채팅 파일 단위테스트 * feat: 파일 도메인 통합테스트 * feat: 파일 도메인 단위테스트 * fix: dto의 필드명은 바뀌지 않도록 롤뱁 * chore: 사용하지 않는 ChatImageRepository 제거 * test: 채팅 도메인 통합 테스트 및 단위 테스트에 이미지 필드 테스트 추가 * fix: 기존 images dto 유지를 위해 필드명 롤백 * fix: 기존 필드명이 imageOrder가 아닌 imgOrder였으므로 롤백 --------- Co-authored-by: dmori <chosehoon2@naver.com> * [docs/#534] 이슈 템플릿 추가 (#535) * [chore/#536] FCM 구현을 위한 firebase 의존성 추가 (#537) * chore: firebase account 설정파일 주입 * chore: ci, cd에 firebase 주입 경로 추가 * chore: build.gradle에 의존성 추가 * fix: 테스트 이슈 라벨링 파싱 문제 해결 * [test/#516] Member 테스트코드 작성 (#533) * feat: Member 통합테스트 구현 * feat: Member 통합테스트 구현 * feat: 모든 필드를 커버하는 통합테스트 추가 * feat: 모든 필드를 커버하는 통합테스트 추가 * test: 중복되는 프로필 테스트 제거 * [test/#538] 운동 신청/취소, 게스트 초대/취소 API 테스트 작성 (#539) * test: 기존에 ExerciseParticipationService에서 Builder 대신 Fixture를 사용하도록 하여 가독성 향상 * test: 나이 조건 테스트를 위해 관련 MemberFixture 추가 * test: 운동 신청 API 단위 테스트 및 통합 테스트 코드 작성 * test: ExerciseCommandService에서 Builder를 쓰던 걸 Fixture를 사용하도록 변경 * test: 운동 신청 API에서 파사드 패턴의 CommandService의 단위 테스트 구현 * test: 운동 신청 취소 API의 통합 테스트 및 단위 테스트 코드 작성 * refactor: 운동 신청, 운동 신청 취소, 게스트 초대, 게스트 초대 취소 API를 ResponseEntity로 응답을 감싸 status도 정상적으로 반환할 수 있도록 변경 * chore: 불필요한 의존성 제거 및 줄바꿈 변경 * test: 게스트 초대 API 통합 테스트 및 CommandService 단위 테스트 코드, GusetService 단위 테스트 코드 작성 * test: 게스트 초대 취소 API 통합 테스트, CommandService 단위 테스트, GuestService 단위 테스트 코드 작성 * [feat/#540] FCM 의존성 추가 및 알림 기능 구현 (#541) * [feat/#540] Firebase 주입방식 변경 (#542) * chore: firebase 주입방식 변경 * chore: 설정파일 추가 * chore: firebase 키 재발급 * feat: firebaseConfig파일 생성 * feat: 설정파일 추가 * feat: FCM 알림 기능 구현 * test: 단위테스트 작성 * chore: valid추가 * test: 통합테스트 config에 mock 추가 * chore: 환경변수 주입 방식으로 변경 * [feat/#540] firebase 환경변수 주입방식으로 변경, docker-compose파일 수정 (#543) * chore: firebase 주입방식 변경 * chore: 설정파일 추가 * chore: firebase 키 재발급 * feat: firebaseConfig파일 생성 * feat: 설정파일 추가 * feat: FCM 알림 기능 구현 * test: 단위테스트 작성 * chore: valid추가 * test: 통합테스트 config에 mock 추가 * chore: 환경변수 주입 방식으로 변경 * chore: docker-compose 파일 수정 * [debug/#544] 모임 추천 조회 (콕플 추천 모드) 검색 기능 누락 해결 * [debug/#547] 프로필 이름 변경 시 내 채팅방 목록이 전부 내 이름으로 변경되는 버그 (#548) * debug: 프로필 이름 변경 시 상대방의 채팅방 목록에서 내 이름이 변경되도록 버그 수정 * test: 프로필 수정 메서드에서 채팅방 이름 변경 로직 단위 테스트 작성 * improve: 통합 테스트에서 실제 스토리지에 데이터를 넣는 건 적절하지 않으므로 mock과 stub으로 대체 * test: 프로필 수정 API 통합 테스트 작성 * [feat/#546] Flyway 도입을 위한 설정 추가 (#549) * chore: flyway설정파일 추가 * chore: flyway 설정 추가 * chore: test에서도 flyway설정이 되도록 수정 --------- Co-authored-by: Dmori <83327857+Dimo-2562@users.noreply.github.com> Co-authored-by: Yumin Kwon <dbalsrltk@naver.com> Co-authored-by: dmori <chosehoon2@naver.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
❤️ 기능 설명
swagger 테스트 성공 결과 스크린샷 첨부
연결된 issue
연결된 issue를 자동으로 닫기 위해 아래 {이슈넘버}를 입력해주세요.
close #521
🩷 Approve 하기 전 확인해주세요!
✅ 체크리스트