You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@toss/date 에서 제공하는 getDateDistance 함수의 역할이 date-fns 라이브러리에서 제공하는 intervalToDuration 함수와 완전히 같습니다. @toss/date는 내부적으로 date-fns에 의존하므로, 코드의 중복을 줄이기 위해 다음 개선을 제안합니다.
개선 방안
라이브러리 이용자가 getDateDistance 함수 대신 intervalToDuration을 사용하도록 유도합니다.
getDateDistanceText 대신 date-fns의 formatDuration을 대체하는 kstFormatDuration을 새로 만듭니다.
유의할 점
date-fns의 formatDuration에는 토스 버전의 함수와 달리, 각 unit을 보여줄지 여부를 계산하는 함수를 매개변수로 받을 수 있는 로직(이하 filter 로직)이 없습니다. 대신 zero, delimiter 등을 매개변수로 설정할 수 있는데, interface extend 방식으로 여기에 filter 매개변수를 추가해 기존 filter 로직을 추가 구현할 수 있습니다.
The text was updated successfully, but these errors were encountered:
링크
@toss/date
에서 제공하는getDateDistance
함수의 역할이date-fns
라이브러리에서 제공하는intervalToDuration
함수와 완전히 같습니다.@toss/date
는 내부적으로date-fns
에 의존하므로, 코드의 중복을 줄이기 위해 다음 개선을 제안합니다.개선 방안
getDateDistance
함수 대신intervalToDuration
을 사용하도록 유도합니다.getDateDistanceText
대신date-fns
의formatDuration
을 대체하는kstFormatDuration
을 새로 만듭니다.유의할 점
date-fns
의 formatDuration에는 토스 버전의 함수와 달리, 각 unit을 보여줄지 여부를 계산하는 함수를 매개변수로 받을 수 있는 로직(이하filter
로직)이 없습니다. 대신 zero, delimiter 등을 매개변수로 설정할 수 있는데, interface extend 방식으로 여기에 filter 매개변수를 추가해 기존 filter 로직을 추가 구현할 수 있습니다.The text was updated successfully, but these errors were encountered: