Skip to content

feat: {프로젝트, 수상, 분과, 공지} 관리 페이지, 마이 페이지, 기타 수정#205

Open
redzzzi wants to merge 101 commits intodevelopfrom
feat/project-manage-admin
Open

feat: {프로젝트, 수상, 분과, 공지} 관리 페이지, 마이 페이지, 기타 수정#205
redzzzi wants to merge 101 commits intodevelopfrom
feat/project-manage-admin

Conversation

@redzzzi
Copy link
Collaborator

@redzzzi redzzzi commented Feb 15, 2026

📝 개요

  • {프로젝트, 수상, 분과, 공지} 관리 페이지, 마이 페이지 기능 개발 및 기존 기능 수정

✨ 변경 사항

  • 프로젝트 관리 페이지 (pages/admin/project-manage/ProjectManagePage.tsx)
    • API 연결 확인 필요
image
  • 수상 관리 페이지 (pages/admin/award-manage/AwardManagePage.tsx)
    • API 연결 확인 필요
image
  • 분과 관리 페이지 (pages/admin/track-manage/TrackManagePage.tsx)
    • API 연결 확인 ✅
image
  • 공지 관리 페이지 (pages/admin/notice-manage/NoticeManagePage.tsx)
    • API 연결 확인 ✅
    • NoticeCreateTab의 타입을 주어 편집 및 생성
image
  • 마이 페이지 (pages/me/MyPage.tsx)
    • 회원 탈퇴 훅 작성
image

🔬 리뷰 요구 사항 (선택)

  • 아이디 관련 훅: useId.ts 파일 훅에서 필수값인 contestId, teamId 등 매번 null 체크하는 것이 중복될 것 같아서, null인 경우에 리다이렉트 되도록하는 use{Contest, Team}IdOrRedirect 훅을 작성했는데 이렇게 사용하는 것에 대해 의견을 여쭙고 싶습니다!
    image

  • 공용 모달 구현 여부: 관리자 페이지에서 각 기능별 사용되는 모달들이 흩어져있어서, 공용 모달 컴포넌트를 구현하는 방향으로 개선하는 것이 좋을지, 아니면 현재처럼 도메인별로 독립적으로 유지하는 것이 좋을지 의견을 여쭙고 싶습니다!

  • 수상 설정 기능: (AwardMangePage.tsx) 팀 조회 API는 정상 동작하는 것을 확인하기 전입니다. 비즈니스 로직이나 UI 측면에서 수정이 필요한 부분이 있을지 미리 의견 주시면 감사하겠습니다!

    • 동작 흐름:
      1. useContestIdOrRedirectcontestId를 가져온 뒤, useTeamList, useAwardViewAdmin으로 팀/수상 정보 로딩
      2. 상단 폼에서 팀 선택 후 수상 수정 (팀 선택하지 않으면 전체 팀 조회 API를 이용하여 전체에 대한 정보 보여줌)
    • 리뷰 파트 상세
      • AwardList에서 필터링 후 렌더링하는 구조가 괜찮은지
      • 등록된 팀이 없을 때, 폼을 아예 렌더링하지 않고 팀 등록 요구하는 문구를 띄우는 것이 나을지

💬 논의 사항 (선택)

  • 현재 페이지별 컴포넌트들이 @pages/<각 페이지별 전용 폴더>에 간혹 속해 있는데, @pages/ 경로에는 페이지 컴포넌트와 섹션 정도만 두고 나머지는 @features/ 폴더로 따로 관리하는 건 어떨까요?
    • 예:
      • 현재: @pages/project-editor/{ProjectEditorPage.tsx, IntroSection.tsx, UrlInput.tsx, ...}
      • 변경: @pages/project-editor/{ProjectEditorPage.tsx, IntroSection.tsx}, @features/project-editor/{UrlInput.tsx, ...}

- 프로젝트 관리 페이지의 분과 표시에 사용
- 공지사항 관리 페이지의 생성 일시 표시에 사용
@chatgpt-codex-connector
Copy link

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

Copy link
Collaborator

@koty08 koty08 left a comment

Choose a reason for hiding this comment

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

작업 수고하셨습니다!! 👍 API 연결 이후 추가로 검토하도록 하겠습니다.

다음은 논의하고자 하는 부분들에 대한 저의 의견입니다.

  • 공용 모달 구현 여부
    -> 전체적인 작업이 끝난 이후 모달 사용되는 부분들을 확인하여, 디자인을 통일 시킬 수 있는 경우 리팩토링 하는 것이 좋을 것 같습니다

  • 수상 설정 기능
    -> 전체적인 구조는 괜찮으나, 페이지 설명에 (AdminHeader에 포함된 부분) 팀을 선택하고 수상을 선택하라는 문구가 있으면 좋을 것 같습니다!

  • @features 폴더 분리 관련
    -> 만약 features로 분리한다면 Section 컴포넌트들도 다 옮겨서, pages 폴더에는 Page 파일만 포함하여 전체적인 라우팅을 한눈에 확인하는 용도로 분리하는 것이 좋을 것 같습니다.

Copy link
Collaborator

Choose a reason for hiding this comment

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

대회 도메인 자체가 커지는 느낌이 있어, 분과, 수상 API는 contests에 작성하는 것보다 별도의 파일로 생성해서 관리하는게 어떨까요?

Copy link
Collaborator

Choose a reason for hiding this comment

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

색인 느낌이 나는 index.ts라는 파일 명칭보다는 공용 상수이기 때문에 common.ts 같은 네이밍이 좋을 것 같습니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

해당 파일에는 base URL만 포함되어 있어서, config.tsenv.ts은 어떨까요?

Copy link
Collaborator

Choose a reason for hiding this comment

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

대회 공지사항도 전체 공지사항이랑 동일하게 모달로 생성/수정 관리하는 것은 어떨까요?

@koty08
Copy link
Collaborator

koty08 commented Feb 18, 2026

아 추가로 현재 merge 대상 브랜치가 main으로 설정되어 있는데 develop으로 변경 부탁드립니다!

@redzzzi redzzzi changed the base branch from main to develop February 18, 2026 12:10
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.

2 participants

Comments