Closed baldwinIV closed 1 year ago
ChartOption
ChartRenderOption
CandlePeriod
candlePeriod
onMessage
market
주석은 제거되어 PR됨
코멘트 피드백 반영 마치고 제가 머지시키도록 하겠습니다.
@kongyb @sronger 말씀해주신 사항 반영 완료되었습니다.
개요
7 에서 구현한 차트에 옵션으로 몇분봉을 렌더링할지 선택할 수 있게 작업했습니다.
작업사항
ChartOption
과ChartRenderOption
을 하나로 합침ChartRenderOption
과CandlePeriod
를 분리candlePeriod
를 리터럴 타입화, 제너릭 활용하여 리터럴 타입을 매핑하여 객체 생성 및 사용 (client/types/ChartTypes.ts)생각해볼점
onMessage
함수를 갈아끼우는 지금의 구현방식이 best practice인지?market
옵션도 변경하며 렌더할때는 socket을 다시 connect하게 되는데, 여기서 벌어지는 error를 어떻게 해결할지 잘 모르는 상태,market
이 변경될때는 페이지도 reload되기 때문에 상관없는데, SPA구현일때는 어떻게 해결해야할지?이미지
주석은 제거되어 PR됨