Skip to content

Conversation

@BHyeonKim
Copy link
Contributor

📝 요약(Summary)

실시간 캔들차트 중복데이터 패칭 수정

VisibleLogicalRangeChange이벤트를 사용하여 무한스크롤을 구현하였는데, 이전에 서버에 요청한 값을 중복으로 요청하는 문제가 발생했습니다.

이를 해결하기 위해 이전에 요청했던 기준 날짜(커서)를 저장하고 이를 사용해서 중복 요청을 막았습니다.

if (
	prevRequestDate.current &&
	prevRequestDate.current <= firstData.time
)
	return;
prevRequestDate.current = firstData.time;

캔들 차트 및 호가창 스타일 수정

  • 호가창의 백그라운드 색상 제거
  • 호가창 애니메이션 제거
  • 호가창 가격 format 적용
  • 캔들차트 툴팁 날짜 format 적용
  • 캔들차트 최소 높이 적용 (모바일 화면에서 작은 높이를 가지는 문제 수정)

🛠️ PR 유형

어떤 변경 사항이 있나요?

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

💬 공유사항 to 리뷰어

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

📸스크린샷 (선택)


✅ PR Checklist

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

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

@BHyeonKim BHyeonKim self-assigned this Jun 24, 2025
@BHyeonKim BHyeonKim merged commit e273f4a into main Jun 24, 2025
2 checks passed
@investfuture-sonarqube1
Copy link

Failed Quality Gate failed

  • 0.00% Coverage on New Code (is less than 80.00%)

Project ID: CleanEngine_cleanengine-fe_c6875537-ed9d-4dfe-b79c-ea5cb6d2a0e3

View in SonarQube

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