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

[feat][#8] 이벤트 관련 API 구현 #9

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

Conversation

mjeongriver
Copy link
Collaborator

@mjeongriver mjeongriver commented Mar 1, 2025

관련 이슈

resolve : #8

작업 내용

사용자 + 관리자 공통 API (이벤트)

  • GET /events → 이벤트 목록 조회(상태별 조회 가능, 쿼리 파라미터로 상태 전달)
  • GET /events/{id} → 이벤트 상세 조회

관리자 전용 API (이벤트)

  • POST /events → 이벤트 등록
  • PATCH /events/{id} → 이벤트 수정
  • DELETE /events/{id} → 이벤트 삭제
  • PATCH /events/{id}/status → 이벤트 상태 변경

이슈

이벤트가 계속 추가될 가능성을 고려해, 현재 이벤트 정책 관리를 Policy 패키지로 분리해두었습니다.
아직 정책 관리 방식에 대해 명확한 방향이 정해지지 않아 다른 도메인 작업 이후 다시 작업 할 예정입니다.

참고 사항

  • feature/5-product-crud에서 작업했던 공통 클래스(APIResponse 등)를 사용하기 위해 해당 브랜치의 내용을 이어와서 작업했습니다.
  • 이번 PR에서는 event 패키지 관련 코드만 리뷰해주시면 감사하겠습니다!

mjeongriver and others added 30 commits February 23, 2025 22:29
Copy link

sonarqubecloud bot commented Mar 2, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
3.1% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@mjeongriver mjeongriver marked this pull request as ready for review March 2, 2025 12:53
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.

[Feat] 이벤트 API 구현
1 participant