Skip to content

♻️ Consolidate multiple event detail pages into a single route#45

Merged
young-52 merged 18 commits intomainfrom
refactor/event-detail
Feb 4, 2026
Merged

♻️ Consolidate multiple event detail pages into a single route#45
young-52 merged 18 commits intomainfrom
refactor/event-detail

Conversation

@jun-0411
Copy link
Collaborator

@jun-0411 jun-0411 commented Feb 4, 2026

📝 작업 내용

  • 분리된 일정 상세 페이지들을 하나로 합쳤습니다.
  • type 이름을 수정하고, 관련된 컴포넌트의 타입 이름도 수정했습니다.
  • 일정 삭제 api와 regId로 유저 정보를 받아오는 api를 구현했습니다.
  • 아직 디자인 수정은 반영하지 않았습니다.
  • api에서 오류 코드 별 오류 처리 부분을 아직 구현하지 않았습니다.

📸 스크린샷 (선택)

🚀 리뷰 요구사항 (선택)

@jun-0411 jun-0411 linked an issue Feb 4, 2026 that may be closed by this pull request
@young-52 young-52 self-requested a review February 4, 2026 08:50
@young-52 young-52 self-assigned this Feb 4, 2026
@young-52 young-52 added the enhancement New feature or request label Feb 4, 2026
@young-52 young-52 added this to the Sprint 5 milestone Feb 4, 2026
Copy link
Collaborator

@young-52 young-52 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

더하여 아래 사항을 추가로 수정하였습니다.

  • 일부 타입을 수정하였습니다.
    • 소셜 로그인 시 AuthProviderGOOGLE만 남겨 놓았습니다.
    • JoinEventResponse 타입은 노션의 API 명세를 참고하여 수정하였습니다.
    • 모집 시작·마감시각, 정원은 제8차 회의에서 null이 아닌 것으로 합의하였습니다. 노션에는 부분적으로 반영되지 않은 상태입니다.
  • 타입 수정 과정에서 수반된 오류를 해결하였습니다.
  • EventPageEventMain
    • 다른 라우트 이름에는 Page라는 단어를 쓰지 않았는데 일정 화면에만 Page를 사용하는 것이 어색하여, 일정 메인 화면이라는 의미로 EventMain이라는 이름으로 바꾸었습니다.

@young-52 young-52 merged commit c16ab32 into main Feb 4, 2026
1 check passed
@young-52 young-52 deleted the refactor/event-detail branch February 4, 2026 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

♻️ Consolidate multiple event detail pages into a single route

2 participants