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] 성적 통계 UI #96

Merged
merged 7 commits into from
Feb 18, 2025
Merged

[Feat] 성적 통계 UI #96

merged 7 commits into from
Feb 18, 2025

Conversation

yongaricode
Copy link
Member

@yongaricode yongaricode commented Feb 18, 2025

🔥 Issues

#52

✅ What to do

  • 성적 통계 UI
  • 성적 추가하기 UI
  • 시험 추가하기 모달
  • 성적/시험이 없는 경우 처리

📄 Description

상세 설명

🤔 Considerations

  • 차트: 데이터 시각화를 위해 recharts를 사용하여 차트를 구현
  • 표: tanstack table을 활용하여 데이터 테이블을 구성
    • 표의 border-radius가 적용되지 않는 문제 발생
    • border-separate를 사용해 radius를 적용했으나, 셀 간 border가 끊겨 보이는 문제 발생
      • 표를 <div>로 감싸고 overflow-hidden 적용
      • table에 border-collapse를 추가하여 radius 적용과 border 연결을 동시에 해결

🛠️ Improvements to Make

  • 아무 시험이 등록되지 않았을 떄 수정
  • 백에서 api 명세서 나오면 맞게 수정해야한다.
  • 통계 차트 불러오는게 너무 느려서... 일단 백과 연동해보고 너무 느리면 로딩 페이지 추가해야할듯

👀 References

스크린샷 또는 참고사항

@yongaricode yongaricode requested a review from ahnsui as a code owner February 18, 2025 13:49
@yongaricode yongaricode changed the base branch from main to develop February 18, 2025 13:49
@yongaricode yongaricode merged commit c6cf20d into develop Feb 18, 2025
2 checks passed
@yongaricode yongaricode added the ✨ Feat 새로운 기능을 추가할 경우 label Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feat 새로운 기능을 추가할 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant