Skip to content
This repository was archived by the owner on Jul 11, 2026. It is now read-only.

feat: show detail score#183

Merged
bae080311 merged 8 commits into
developfrom
feat/showScore
Jul 7, 2025
Merged

feat: show detail score#183
bae080311 merged 8 commits into
developfrom
feat/showScore

Conversation

@bae080311

Copy link
Copy Markdown
Member

📝작업 내용

상세 점수보기 기능을 구현했습니다

@bae080311 bae080311 self-assigned this Jul 6, 2025
@bae080311 bae080311 added the 🌟 Admin Admin 페이지와 관련된 이슈를 작성해주세요! label Jul 6, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

Implements a detailed score-viewing feature by making list titles optional, adding a score modal and associated data fetching, and integrating the modal into the member information UI.

  • Made List component’s title prop optional and updated rendering behavior.
  • Created ScoreModal component, useGetScore hook, and API call to fetch scores.
  • Integrated modal open/close logic into Information UI and wired up modal display.

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
packages/shared/src/list.tsx Made title optional and defaulted missing title
apps/admin/src/widgets/member/ui/information.tsx Added modal state, handlers, and modal integration
apps/admin/src/entities/member/ui/scoreModal.tsx Implemented score modal with data fetching
apps/admin/src/entities/member/model/useGetScore.ts Added React Query hook to fetch scores
apps/admin/src/entities/member/model/score.ts Defined score-related interfaces
apps/admin/src/entities/member/api/getScore.ts Implemented API call with error handling
Comments suppressed due to low confidence (2)

apps/admin/src/entities/member/ui/scoreModal.tsx:1

  • No tests cover the ScoreModal component; consider adding unit or integration tests to validate modal rendering and data fetching behaviors.
"use client";

apps/admin/src/widgets/member/ui/information.tsx:57

  • [nitpick] Slicing the ID starting at index 1 removes the first character of the username; verify this is intentional or adjust the slice range to include the full identifier.
        id={student?.email?.split("@")[0]?.slice(1) as string}

Comment thread packages/shared/src/list.tsx
Comment thread apps/admin/src/entities/member/ui/scoreModal.tsx Outdated
Comment thread apps/admin/src/entities/member/ui/scoreModal.tsx Outdated
@github-actions

github-actions Bot commented Jul 6, 2025

Copy link
Copy Markdown

✅ CI success

@bae080311 bae080311 changed the title Feat/show score feat: show detail score Jul 6, 2025
@github-actions

github-actions Bot commented Jul 6, 2025

Copy link
Copy Markdown

✅ CI success

Comment thread apps/admin/src/entities/member/api/getScore.ts
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@github-actions

github-actions Bot commented Jul 7, 2025

Copy link
Copy Markdown

✅ CI success

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@github-actions

github-actions Bot commented Jul 7, 2025

Copy link
Copy Markdown

✅ CI success

@bae080311
bae080311 merged commit 001a137 into develop Jul 7, 2025
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

🌟 Admin Admin 페이지와 관련된 이슈를 작성해주세요!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants