Skip to content

Conversation

@BHyeonKim
Copy link
Contributor

📝 요약(Summary)

main 브랜치에서 분기한 이후 develop 브랜치에서 진행된 대규모 업데이트를 병합합니다. 사용자 프로필 및 거래 내역 기능, 포괄적인 테스트 인프라 구축, 호가창 개선, 에러 핸들링 강화, 그리고 다양한 UI/UX 개선사항이 포함되어 있습니다. (총 83개 커밋)

🛠️ PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 문서 수정
  • 파일 혹은 폴더 수정/삭제

주요 기능 추가

  • 프로필 시스템: 사용자 자산 조회, 체결 내역, 파이차트 시각화
  • 거래 내역 관리: 페이지네이션, 주문 취소, 에러 핸들링
  • UI 컴포넌트: Tab, Pagination, IncrementingNumber, Error, Spinner 등
  • 호가창 개선: AmCharts에서 Recharts로 마이그레이션, 정렬 기능

테스트 인프라

  • 단위 테스트: StompProvider, UserInfoProvider, 각종 컴포넌트
  • API 테스트: history 엔드포인트, MSW 목업 데이터
  • 훅 테스트: useCurrentPrice, 커스텀 훅들
  • 컴포넌트 테스트: AIChatBot, CoinPriceWithName, CoinWithIconAndName

버그 수정 및 개선

  • 인증 시스템: 로그인 상태 관리, 쿠키 기반 인증
  • 라우팅: SSR/CSR 전환 문제, 에러 바운더리
  • 데이터 일관성: 코인 가격 표시 오류, 시간대 계산
  • 사용자 경험: 스크롤 동작, 모달 인터랙션, 로딩 상태

💬 공유사항 to 리뷰어

  • 이번 PR은 83개의 커밋이 포함된 대규모 업데이트입니다. 주요 기능별로 나누어 리뷰해주시면 감사하겠습니다.
  • 호가창이 AmCharts에서 Recharts로 완전히 마이그레이션되었습니다. 기능 동등성과 성능을 확인해주세요.
  • 새로 추가된 테스트들이 충분한 커버리지를 제공하는지, 그리고 테스트 시나리오가 적절한지 검토 부탁드립니다.
  • 프로필 관련 기능들의 UX가 직관적인지, 특히 파이차트와 테이블 간의 인터랙션이 자연스러운지 확인해주세요.

📸스크린샷 (선택)


✅ PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

BHyeonKim added 30 commits June 29, 2025 22:06
code routes가 전체적인 routes 파악과 유지보수가 쉽다는 생각이 들어서 code routes로 변경하였습니다.
코인 자산 비율을 확인할 수 있는 파이 차트 컴포넌트를 추가했습니다.
Sentry 무료 플랜이 끝나서 Sentry 설정을 주석처리하였습니다.
나중에 Sentry를 onpremise로 구축해 주석 해제 예정입니다.
layout shifting이 일어나서 스트리밍을 제거하고 ssr로 변경했습니다.
TradingHistoryList에서 페이지네이션 상태를 URLSearchParam을 조작하여 상태를 저장하고 있는데
해당 기능은 navigation을 유발합니다.
따라서 페이지 이동후 모달 외부를 클릭했을 때 모달이 닫히는것이 아닌
이ㅣ전 페이지로 돌아가므로 이를 수정하였습니다.
BHyeonKim and others added 29 commits July 27, 2025 00:19
feat: 매도 호가 가격 내림차순 정렬로 변경
렌더링 후에 웹소켓 커넥션이 연결되고 실시간 가격이 올때까지 rest api로 받은 과거
가격을 보여주도록 하였습니다.

issues: closes #37
@BHyeonKim BHyeonKim merged commit e9757ca into main Aug 2, 2025
0 of 2 checks passed
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.

3 participants