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

[Feature]:@toss/dategetDateDistance 관련 함수를 date-fns 라이브러리와 호환시키기 #435

Open
CodePsy-2001 opened this issue Feb 23, 2024 · 0 comments

Comments

@CodePsy-2001
Copy link

링크

@toss/date 에서 제공하는 getDateDistance 함수의 역할이 date-fns 라이브러리에서 제공하는 intervalToDuration 함수와 완전히 같습니다.
@toss/date는 내부적으로 date-fns에 의존하므로, 코드의 중복을 줄이기 위해 다음 개선을 제안합니다.

개선 방안

  • 라이브러리 이용자가 getDateDistance 함수 대신 intervalToDuration을 사용하도록 유도합니다.
  • getDateDistanceText 대신 date-fnsformatDuration을 대체하는 kstFormatDuration을 새로 만듭니다.

유의할 점

  • date-fns의 formatDuration에는 토스 버전의 함수와 달리, 각 unit을 보여줄지 여부를 계산하는 함수를 매개변수로 받을 수 있는 로직(이하 filter 로직)이 없습니다. 대신 zero, delimiter 등을 매개변수로 설정할 수 있는데, interface extend 방식으로 여기에 filter 매개변수를 추가해 기존 filter 로직을 추가 구현할 수 있습니다.
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

No branches or pull requests

1 participant