Refactor/remove legacy codes - #6
Merged
Merged
Conversation
- 사용하지 않는 ChatRoom, ChatMessage 엔티티 및 관련 로직 제거
- 중복 체크 및 보호자 매핑 로직을 별도 메서드로 분리하여 가독성 개선
- 예외 처리를 LifelineException(ErrorCode)으로 통일 - 반복되는 프로필 조회 로직을 Helper Method로 추출
…to refactor/remove-legacy-codes
-Lombok의 @builder 사용 시 alarms 리스트의 초기화 식(new ArrayList<>())이 무시되는 문제 해결
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
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.
⏰ Summary
빌드 경고 해결 (AlarmGroup): @builder 사용 시 리스트 초기화 식이 무시되던 문제를 @Builder.Default 어노테이션 추가를 통해 해결하고 빌드 경고를 제거
환경 변수 안전장치 추가 (AuthSmsService): CoolSMS 발신 번호(SMS_FROM_NUMBER) 등 환경 변수가 없는 환경에서도 테스트가 중단되지 않도록 @value에 기본값을 설정
ApiResponse 구조 개선: 제네릭 타입 추론 이슈와 JSON 이중 배열 문제를 해결하기 위해 정적 팩토리 메서드(ok())를 도입
CI/CD 환경 최적화: GitHub Actions의 빌드 실패 문제를 해결하기 위해 필수 환경 변수들을 GitHub Secrets에 모두 등록 완료
[ 클래스별 상세 변경 내역 ]