feat: 비즈니스 예외와 API 공통 응답 모듈 추가 by mandykr · Pull Request #23 · Team-Retrip/crew

mandykr · 2025-02-25T05:44:14Z

🔍 PR 타입 선택

아래 타입 중 해당하는 하나를 선택해 주세요. 반드시 하나만 선택해 주세요.

feat: 새로운 기능 추가

📝 변경 사항 요약

변경 사항을 간단히 요약해 주세요.

비즈니스 예외와 API 공통 응답 모듈 추가
vo 검증에서 발생하는 비즈니스 예외 삭제 및 InvalidValueException 으로 변경

🛠 관련 이슈

Resolves: #22

추가 설명 (선택 사항)

변경 사항에 대한 추가 설명을 작성해 주세요.

- vo 검증에서 발생하는 비즈니스 예외 삭제 및 InvalidValueException 으로 변경

src/main/java/com/retrip/crew/domain/exception/CrewNotFoundException.java

src/main/java/com/retrip/crew/domain/exception/common/BusinessException.java

src/main/java/com/retrip/crew/domain/exception/common/ErrorCode.java

src/main/java/com/retrip/crew/infra/adapter/in/presentation/rest/CrewController.java

xjvmdutl · 2025-02-26T00:56:13Z

src/main/java/com/retrip/crew/infra/adapter/in/presentation/rest/common/ErrorResponse.java

+    private int status;
+    private String code;
+    private String message;
+    private String url = "Not available";


해당 값("Not available")을 디폴트 값을 설정한 이유가 있을까요?

응답해줄 url이 없을때 비어있는 값보다 더 명확한 의미를 나타내게 하려고 했습니다

* feat: 비즈니스 예외와 API 공통 응답 모듈 추가 - vo 검증에서 발생하는 비즈니스 예외 삭제 및 InvalidValueException 으로 변경

* feat: PR 템플릿 and PR Discord 알림 연동 * feat: Crew 생성 (#20) * feat: Crew 생성 * feat: Code 리뷰 반영, Exception 추가 * feat: 주석 제거 및 코드 수정 --------- Co-authored-by: junhokim <junhokim@gmarket.com> * feat: 비즈니스 예외와 API 공통 응답 모듈 추가 (#23) * feat: 비즈니스 예외와 API 공통 응답 모듈 추가 - vo 검증에서 발생하는 비즈니스 예외 삭제 및 InvalidValueException 으로 변경 * feat: 스웨거 추가 작업 (#24) Co-authored-by: junhokim <junhokim@gmarket.com> * [feat] 크루 상세, 리스트 조회 API 기능 구현 (#25) * [feat] 크루 리스트 조회 API 기능 구현 및 테스트코드 리팩토링 (#4) * [feat] 크루 상세 조회 API 기능 구현 (#4) * feat: 스웨거 추가 작업 (#24) Co-authored-by: junhokim <junhokim@gmarket.com> * [fix] 크루 상세 조회 API 코드리뷰 반영 (#4) * [fix] 단위테스트 SpringBootTest -> DataJpaTest 사용하도록 수정 (#4) --------- Co-authored-by: Kimjunho <39546306+xjvmdutl@users.noreply.github.com> Co-authored-by: junhokim <junhokim@gmarket.com> * 크루 참여 요청 (#29) * feat: 크루 모집상태 변경시 최대 인원수 검증 추가 - IllegalStateException 추가 - common exception 생성자 수정 * refactor: CrewMembers size 조회 메서드 추가 * fix: conflict repair * feat:크루 참여 요청 질문 등록 * feat:크루 참여 요청 질문 등록 * [FEAT]-recruitment로 연관관계 변경 * 크루 참여 요청 질문 등록 수정 * 크루 참여 요청 질문 등록 수정 구현 --------- Co-authored-by: junhokim <junhokim@gmarket.com> Co-authored-by: Kimjunho <39546306+xjvmdutl@users.noreply.github.com> Co-authored-by: mandykr <94955454+mandykr@users.noreply.github.com> Co-authored-by: coPpark <qkrwjdtn963@gmail.com> Co-authored-by: MonChest <p_caso@atzsoft.kr>

feat: 비즈니스 예외와 API 공통 응답 모듈 추가

dbbb8af

- vo 검증에서 발생하는 비즈니스 예외 삭제 및 InvalidValueException 으로 변경

mandykr added the Priority: High high label Feb 25, 2025

mandykr added this to the Sprint 1 milestone Feb 25, 2025

mandykr requested review from ekfrehd, jisubaek, pparkjs and xjvmdutl February 25, 2025 05:44

mandykr self-assigned this Feb 25, 2025

mandykr linked an issue Feb 25, 2025 that may be closed by this pull request

비즈니스 예외와 API 공통 응답 모듈 설계 #22

Closed

3 tasks