[feat] preMember 목록 분리 및 삭제 기능 추가

[ff1451]

Summary by CodeRabbit

• New Features
  • Manager: view pending (pre-)members in a dedicated list.
  • Manage pre-members via an action dialog and a delete confirmation modal.
  • Delete pending members with success feedback and automatic list refresh.
  • Background fetch/delete hooks to keep the Manager dashboard up to date.

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Adds PreMember-related types, two club API endpoints (getPreMembers, deletePreMember), React Query hooks to fetch/delete pre-members with cache invalidation, and UI support in the manager page (pre-member list, action modal, delete-confirm modal, deletion flow with toast).

Changes

Cohort / File(s)	Summary
API Entities src/apis/club/entity.ts	Added PreMember, PreMembersList, and PreMemberDeleteRequest interfaces.
Club API src/apis/club/index.ts	Added getPreMembers(clubId): PreMembersList and deletePreMember(clubId, preMemberId): void endpoints (requiresAuth).
Manager Hooks src/pages/Manager/hooks/useManagerQuery.ts	Added useGetPreMemberList(clubId) and useDeletePreMember(clubId, options?); added query key preMembersList and invalidate it on relevant mutations.
Manager UI / Page src/pages/Manager/ManagedMemberList/index.tsx	Integrated pre-member list rendering, selection state, action modal and delete-confirm modal, wired delete mutation with close/clear/toast behavior; minor input digit validation.

Sequence Diagram

sequenceDiagram
    participant User as User
    participant UI as ManagedMemberList
    participant Hook as useDeletePreMember
    participant API as Club API
    participant Cache as ReactQuery Cache

    User->>UI: Click "Manage" on a pre-member → choose "Delete"
    UI->>User: Show delete confirmation modal
    User->>UI: Confirm deletion
    UI->>Hook: mutateDeletePreMember(clubId, preMemberId)
    Hook->>API: DELETE /clubs/{clubId}/pre-members/{preMemberId}
    API-->>Hook: 200 OK
    Hook->>Cache: invalidateQuery("preMembersList", clubId)
    Cache-->>Hook: refetch pre-members
    Hook-->>UI: mutation success
    UI->>UI: Close modals, clear selection, show toast

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related issues

• [feat] pre-member 목록 분리 및 API 적용 #111: Adds the same pre-member list management (types, APIs, hooks, UI) described in the issue.

Possibly related PRs

• [feat] preMember 목록 분리 및 삭제 기능 추가 #115: Implements the same PreMember interfaces, club API functions, and manager hooks/UI for pre-member fetch/delete.

Poem

🐰 I nibble at types and hop through hooks,
A pre-member list tucked in tidy books.
A modal, a click, then a soft goodbye,
Cache refreshed beneath a springtime sky.
Hooray — carrot-coded joy, up high!

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title in Korean describes adding pre-member list separation and deletion functionality, which directly matches the main changes: new PreMember entities, API functions (getPreMembers, deletePreMember), hooks, and UI implementation for managing pre-members.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch 111-feat-pre-member-목록-분리-및-api-적용

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

🧹 Nitpick comments (1)

src/apis/club/entity.ts (1)

270-285: Use PascalCase for exported interfaces.

These new exported types are lower camelcase while the rest of the file uses PascalCase. Renaming keeps the domain model consistent and avoids lint/style drift. Please update usages in the API layer and UI imports accordingly.

🔁 Suggested rename (update usages accordingly)

-export interface preMember {
+export interface PreMember {
   preMemberId: number;
   studentNumber: string;
   name: string;
   clubPosition: 'MANAGER' | 'MEMBER';
 }
 
-export interface preMembersList {
-  preMembers: preMember[];
+export interface PreMembersList {
+  preMembers: PreMember[];
 }
 
-export interface preMemberDeleteRequest {
+export interface PreMemberDeleteRequest {
   clubId: number;
   preMemberId: number;
 }

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/apis/club/entity.ts` around lines 270 - 285, The exported interfaces
preMember, preMembersList, and preMemberDeleteRequest use lower camelcase which
is inconsistent with the PascalCase naming convention used elsewhere. Rename
these interfaces to PreMember, PreMembersList, and PreMemberDeleteRequest
respectively, and ensure all usages in the API layer and UI imports are updated
to match these new names for consistency across the domain model.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@src/apis/club/entity.ts`:
- Around line 270-285: The exported interfaces preMember, preMembersList, and
preMemberDeleteRequest use lower camelcase which is inconsistent with the
PascalCase naming convention used elsewhere. Rename these interfaces to
PreMember, PreMembersList, and PreMemberDeleteRequest respectively, and ensure
all usages in the API layer and UI imports are updated to match these new names
for consistency across the domain model.