Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[♻️ refactor] 약속 생성 API 패키징 개선 #81

Open
wants to merge 23 commits into
base: develop
Choose a base branch
from

Conversation

jsoonworld
Copy link
Member

🚀 What’s this PR about?

  • 작업 내용 요약: 약속 생성 API의 패키징을 개선하였습니다.

🛠️ What’s been done?

  • 주요 변경사항:
    • RESTful API에 맞게 약속 생성 관련 클래스를 적절한 패키지로 이동하였습니다.

🧪 Testing Details

  • 테스트 코드 및 결과:
    • 모든 테스트가 정상적으로 동작함을 확인하였습니다.

👀 Checkpoints for Reviewers

  • 리뷰 시 확인할 사항:
    • 패키지 구조가 적절하게 이동되었는지 확인 부탁드립니다.
    • 개선된 URL 구조에 대한 의견이 있으시면 공유해 주세요.

🎯 Related Issues

- `@RequestMapping("/api/v1/groups/{groupId}/appointments")`와 `@RequestMapping("/api/v1/appointments")` 중복 제거
- API 경로의 일관성을 유지하도록 수정
- `org.noostak.appointment.application` → `org.noostak.group.application.create` 패키지로 이동
- 도메인별 역할을 명확히 구분하기 위해 구조 개편
- `org.noostak.appointment.application` → `org.noostak.group.application.create` 패키지로 이동
- 그룹 도메인 내에서 관리하는 것이 적절하다고 판단하여 구조 변경
- 예외 처리 시 `AppointmentException` → `GroupException`으로 변경하여 일관성 유지
- `AppointmentCreateService`의 패키지가 변경되어, `org.noostak.group.application.create`로 수정
- `AppointmentCreateRequest` DTO의 패키지 변경에 따라 import 경로 수정
- `GROUP_NOT_FOUND` 오류 메시지에 대한 수정
- 그룹과 관련된 에러 코드 `GROUP_REQUIRED`, `MEMBER_NOT_IN_GROUP` 추가
- 약속 이름 유효성 검사를 위한 에러 코드 추가
- 약속 생성 및 확정 관련 성공 메시지 추가
- `HttpStatus`를 활용하여 HTTP 응답 상태 코드와 함께 성공 메시지 설정
- `org.noostak.appointment.dto.request` → `org.noostak.group.dto.request` 패키지로 이동
- 그룹 관련 기능으로 이동하여 도메인 구조를 명확히 함
- `org.noostak.appointment.dto.request` → `org.noostak.group.dto.request` 패키지로 이동
- 그룹 관련 기능으로 이동하여 도메인 구조를 명확히 함
- 기존 `AppointmentController`에서 약속 생성 기능을 `GroupController`로 이동
- 그룹 관련 기능에 맞춰 `AppointmentCreateRequest` 및 `GroupService` 의존성 추가
- API 경로를 `/groups/{groupId}/appointments`로 변경
- `org.noostak.group.application` → `org.noostak.group.application.create` 패키지로 이동
- 그룹 생성 관련 기능을 새로운 패키지로 분리하여 구조 정리
- `org.noostak.group.application` → `org.noostak.group.application.retrieve` 패키지로 이동
- 그룹 조회 기능을 별도 패키지로 분리하여 구조 정리
- `GroupCreateService`, `GroupRetrieveService`, `GroupInfoService`, `AppointmentCreateService` 의존성 주입
- 그룹 생성, 그룹 정보 조회, 약속 생성 기능 통합
- 각 서비스의 메서드를 호출하여 그룹 관련 작업 처리
- `MEMBER_NOT_FOUND` 오류 코드 중복 제거
- 그룹 관련 예외 처리 및 오류 메시지 수정
- `GROUP_INFO` 성공 코드 중복 제거
- 그룹 관련 성공 메시지 수정
@jsoonworld jsoonworld added ♻️ refactor Improvements or additions to documentation 🐋 장순 🐋 This issue or pull request already exists labels Feb 25, 2025
@jsoonworld jsoonworld requested a review from 0-tae February 25, 2025 04:30
@jsoonworld jsoonworld self-assigned this Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor Improvements or additions to documentation 🐋 장순 🐋 This issue or pull request already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant