Skip to content

Pjh#157

Merged
euics merged 5 commits into
masterfrom
develop
Oct 29, 2025
Merged

Pjh#157
euics merged 5 commits into
masterfrom
develop

Conversation

@euics

@euics euics commented Oct 29, 2025

Copy link
Copy Markdown
Member

🌱 관련 이슈

  • close #

📌 작업 내용 및 특이사항

📝 참고사항

📚 기타

euics added 5 commits October 29, 2025 21:45
- 버그 수정: keyword 필터링 조건 누락, ExceptionCodeMapper 버그, ExceptionType 수정
- 코드 개선: for문 → stream, split 로직 개선, 에러 처리 강화
- Repository: @Param 패키지 수정, 쿼리 개선 (정렬, distinct, innerJoin)
- Controller: null 체크 강화, @apiresponse 정확성 개선
- 기타: 불필요한 import 제거, @OverRide 추가, 정렬 추가
- Controller: 메서드명 수정, 변수명 개선, null 체크 강화, 페이지 검증 추가
- Service: @transactional 추가, @transactional(readOnly = true) 추가
- Repository: limit/offset 계산 검증 추가 (음수 방지)
- Service: null 체크 강화, 조건 로직 개선, switch default 추가, 예외 처리 강화
- Repository: join을 innerJoin으로 명시, limit/offset 검증 추가, @Param 패키지 수정
- Controller: null 체크 강화, 예외 처리 개선
- 매직 넘버 상수화 (TAX_RATE)
- 불필요한 import 제거
- Exception 클래스: 코드 포맷팅 개선 (공백 제거)
- ExceptionCodeMapper: 오타 수정 (NOT_FOUNE → NOT_FOUND)
- GlobalExceptionHandler: 상수 분리, ObjectMapper 재사용, 변수명 개선, 타입 안전성 개선
@euics
euics merged commit 199daff into master Oct 29, 2025
2 checks passed
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.

1 participant