fix: amountToHangul이 소수점, 숫자도 대응할 수 있도록 수정 #144

Collection50 · 2024-06-29T13:51:09Z

Overview

amountToHangul의 함수명만 본다면 number | string 모두 변환가능하다고 인식될 수 있습니다.
기존 string만 가능하던 변환을 number로도 가능할 수 있도록 확장했습니다.
소숫점에도 대응 가능할 수 있도록 기능을 확장했습니다.
줄바꿈 등을 수정하여 가독성을 향상했습니다.

AS-IS

amountToHangul('15,201,100'); // '일천오백이십만천백';
amountToHangul('120,030원'); // '일십이만삼십' - 숫자 외 문자 무시
amountToHangul('392.24'); // '삼백구십이' - 소수점 무시

TO-BE

amountToHangul('15,201,100'); // '일천오백이십만천백'
amountToHangul('120,030원'); // '일십이만삼십'
amountToHangul('12345.6789'); // '일만이천삼백사십오점육칠팔구'
amountToHangul(15_201_100); // '일천오백이십만천백''

PR Checklist

I read and included theses actions below

I have read the Contributing Guide
I have written documents and tests, if needed.

changeset-bot · 2024-06-29T13:51:12Z

🦋 Changeset detected

Latest commit: 1020dfd

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

vercel · 2024-06-29T13:51:13Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
es-hangul	❌ Failed (Inspect)			Jun 30, 2024 1:57am

codecov-commenter · 2024-06-29T13:52:53Z

Codecov Report

Attention: Patch coverage is 95.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 98.82%. Comparing base (b0e97d5) to head (511706c).
Report is 7 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #144      +/-   ##
==========================================
+ Coverage   98.72%   98.82%   +0.10%     
==========================================
  Files          14       16       +2     
  Lines         235      255      +20     
  Branches       53       56       +3     
==========================================
+ Hits          232      252      +20     
  Misses          3        3

okinawaa · 2024-06-29T14:15:25Z

소숫점을 다루는것 까지 확장하는것이 좋을지, 의문이네요. 복잡성이 증가하는것에 비해서, 큰 장점이 있다고는 생각이 들지 않아서요!
@crucifyer 님께서, 이 함수를 직접 제작해주셨는데, 혹시 어떻게 생각하시나요~?

crucifyer · 2024-06-29T15:05:57Z

제 원래의 목적은 '돈 세는'(amount) 거라서 소수점을 굳이 추가하지 않았습니다. :)
활용도가 없을 듯 해서 다른 변환방식은 추가하지 않았어요.

amountToHangul('10010') + '원정'

원래 제가 만들었던것은 이러한 기능까지 있습니다.
https://crucifyer.github.io/koreanCardinalOrdinal/

okinawaa

감사합니다!

okinawaa

타입이 변경되기때문에 minor변경으로 처리하겠습니다

* fix: amountToHangul이 소수점, 숫자도 대응할 수 있도록 수정 * docs: amountToHangul의 영어 문서 작성 * Create famous-cheetahs-sneeze.md * Update famous-cheetahs-sneeze.md --------- Co-authored-by: 박찬혁 <pgg6713@gmail.com>

* fix. 겹모음과 관련된 이상 현상들 수정 * fix. fix index * fix. 오타 수정 Co-authored-by: 박찬혁 <pgg6713@gmail.com> * fix. 오타 수정 Co-authored-by: 박찬혁 <pgg6713@gmail.com> * fix. 업데이트 로직 반영 * test: internal 테스트 코드를 작성합니다 (#137) * test: internal 테스트 코드를 작성합니다 * test: vitest에서 제공해주는 toThrowError로 변경 --------- Co-authored-by: 박찬혁 <pgg6713@gmail.com> * docs: josa에 관련된 문서에 잘못된 부분을 수정합니다 - '이에/에' 삭제 Co-authored-by: 박찬혁 <pgg6713@gmail.com> * chore: version packages (#143) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * feat: 한글 문자열에 대한 검증, assert, parsing 함수를 구현합니다. (#136) * feat: 한글 문자열인지 boolean 반환 함수 * feat: 한글 문자열인지 assert 함수 * feat: stringify * test: isHangulString, assertHangulString 테스트 코드 * feat: parse 함수 * test: parse 함수 테스트 * refactor: naming 통일 * feat : 문장의 각 단어 중 첫 문자만 뽑는 함수추가 ( #128 이슈에 대한 ) (#133) * feat : 문장의 각 단어 중 첫 문자만 뽑는 함수추가 * test 및 함수 추가 * add : 한글 문장인지 여부 판별 함수 추가 * fix : 한글 문장인지 여부 판별 기저 및 오류 추가 / arg 이름 변경 test 추가 * fix: src/_internal/hangul.ts Co-authored-by: 박찬혁 <pgg6713@gmail.com> * fix : rename function getFirstHangulLetters -> getHangulAcronym * fix : rename function export function isHangulOnly로 변경 * fix : lint error * fix : index에 추가 * chore : doc 추가 * fix : 문서화 한글 영어 바뀐거 바로 변경 * fix : isHangul로 대체 #136 으로 * chore : doc수정 * Update docs/src/pages/docs/api/getHangulAcronym.en.mdx Co-authored-by: 박찬혁 <pgg6713@gmail.com> * fix : Update hangul.ts 안쓰는 메소드 삭제 * Update getHangulAcronym.ko.mdx * Update getHangulAcronym.en.mdx * Update src/getHangulAcronym.ts Co-authored-by: 박찬혁 <pgg6713@gmail.com> * Update src/getHangulAcronym.ts Co-authored-by: 박찬혁 <pgg6713@gmail.com> * Update getHangulAcronym.ts * Update getHangulAcronym.spec.ts * Create fair-brooms-drive.md --------- Co-authored-by: 박찬혁 <pgg6713@gmail.com> * chore: version packages (#145) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * feat: 문자열에서 한글만 반환하는 extractHangul을 구현합니다. (#130) * feat: parseHangul * fix: parseHangul의 이름을 extractHangul로 수정 및 테스트 코드 보완 * test: 일관된 테스트 코드 작성이 될 수 있도록 describe 설명 수정 * test: 테스트 코드 수정 * docs: extractHangul의 문서 작성 * test: 테스트 문구 수정 * fix: index.ts에 export 추가 * Create fresh-students-sit.md --------- Co-authored-by: 박찬혁 <pgg6713@gmail.com> * chore: version packages (#146) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * fix : getHangulacronym함수를 acronymizeHangul 메서드로 대체합니다. (#148) * Update and rename getHangulAcronym.ts to acronymizeHangul.ts * Update and rename getHangulAcronym.spec.ts to acronymizeHangul.spec.ts * Update index.ts * Update and rename getHangulAcronym.en.mdx to acronymizeHangul.en.mdx * Update and rename getHangulAcronym.ko.mdx to acronymizeHangul.ko.mdx * Create odd-squids-sin.md --------- Co-authored-by: 박찬혁 <pgg6713@gmail.com> * chore: version packages (#150) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * fix: amountToHangul이 소수점, 숫자도 대응할 수 있도록 수정 (#144) * fix: amountToHangul이 소수점, 숫자도 대응할 수 있도록 수정 * docs: amountToHangul의 영어 문서 작성 * Create famous-cheetahs-sneeze.md * Update famous-cheetahs-sneeze.md --------- Co-authored-by: 박찬혁 <pgg6713@gmail.com> * fix: packlint ESLint error (#149) * test: utils 기능들의 테스트 케이스를 개선합니다 (#151) * test: hasBatchim 함수의 테스트 케이스를 추가합니다 * test: hasSingleBatchim의 테스트 케이스를 추가합니다 * fix: nextra theme asset 개선 (#152) * chore: version packages (#153) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * fix: 패키지가 노출하는 인터페이스를 명확히 하기 위해 index.ts를 named export로 수정합니다 (#157) * fix: exposing public apis strictly * Create wild-cows-juggle.md * chore: version packages (#160) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * docs: CONTRIBUTING.md 파일에 메서드 컨벤션을 작성합니다. (#132) * Update CONTRIBUTING.md * fix * write contribution.md Co-authored-by: Jonghyeon Ko <61593290+manudeli@users.noreply.github.com> Co-authored-by: Dongkyu Kim <55759551+po4tion@users.noreply.github.com> * Update .github/CONTRIBUTING.md Co-authored-by: Jonghyeon Ko <manudeli.ko@gmail.com> * Update .github/CONTRIBUTING.md Co-authored-by: Jonghyeon Ko <manudeli.ko@gmail.com> * Update .github/CONTRIBUTING.md Co-authored-by: Jonghyeon Ko <manudeli.ko@gmail.com> --------- Co-authored-by: Jonghyeon Ko <61593290+manudeli@users.noreply.github.com> Co-authored-by: Dongkyu Kim <55759551+po4tion@users.noreply.github.com> Co-authored-by: Jonghyeon Ko <manudeli.ko@gmail.com> * test: amountToHangul 테스트 추가 (#135) * test: amountToHangul 테스트 추가 * test: amountToHangul 테스트 코드 수정 * test: amountToHangul 테스트 코드 스타일 변경 * test: amountToHangul test 개선 * fix: assert.throws대신 vitest toThrow로 메서드 변경 * fix: vitest toThrowError 메서드로 변경 --------- Co-authored-by: kinndohyun <dohyun.kim@carbonsaurus.com> Co-authored-by: 박찬혁 <pgg6713@gmail.com> * fix: 소수점 추가 후 발생한 '영'읽기 버그 수정 (#159) * fix: 소수점 추가 후 발생한 '영'읽기 버그 수정 * Create big-cups-call.md --------- Co-authored-by: 박찬혁 <pgg6713@gmail.com> * docs: acronymizeHangul 리턴 타입 표기 수정 (#161) * chore: version packages (#163) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * fix: 일관된 toThrowError 메서드 사용 (#166) Co-authored-by: kinndohyun <dohyun.kim@carbonsaurus.com> * chore(eslint): 의도치 않은 naming을 제거하기 위해 cspell 추가 (#164) * chore(eslint): add cspell to check bad naming * chore: update * Create grumpy-singers-love.md --------- Co-authored-by: 박찬혁 <pgg6713@gmail.com> Co-authored-by: Dongkyu Kim <po4tion0429@gmail.com> Co-authored-by: Jaemin Kim <jmkim326@gmail.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: seunghee <sa02045@naver.com> Co-authored-by: taehyun <126179088+KNU-K@users.noreply.github.com> Co-authored-by: SeongMin Kim <86355699+Collection50@users.noreply.github.com> Co-authored-by: Jonghyeon Ko <manudeli.ko@gmail.com> Co-authored-by: Jonghyeon Ko <61593290+manudeli@users.noreply.github.com> Co-authored-by: Dongkyu Kim <55759551+po4tion@users.noreply.github.com> Co-authored-by: 김도현 <101170386+fe-dudu@users.noreply.github.com> Co-authored-by: kinndohyun <dohyun.kim@carbonsaurus.com> Co-authored-by: Song Hyo Jin <shj@xenosi.de> Co-authored-by: wnhlee <40269597+2wheeh@users.noreply.github.com>

fix: amountToHangul이 소수점, 숫자도 대응할 수 있도록 수정

ccb23ae

Collection50 requested review from okinawaa and raon0211 as code owners June 29, 2024 13:51

vercel bot had a problem deploying to Preview June 29, 2024 13:51 Failure

docs: amountToHangul의 영어 문서 작성

8cfaf26

vercel bot had a problem deploying to Preview June 29, 2024 13:53 Failure

Create famous-cheetahs-sneeze.md

511706c

vercel bot had a problem deploying to Preview June 30, 2024 01:55 Failure

okinawaa previously approved these changes Jun 30, 2024

View reviewed changes

Update famous-cheetahs-sneeze.md

1020dfd

okinawaa dismissed their stale review via 1020dfd June 30, 2024 01:56

okinawaa approved these changes Jun 30, 2024

View reviewed changes

vercel bot had a problem deploying to Preview June 30, 2024 01:57 Failure

okinawaa merged commit b114897 into toss:main Jun 30, 2024
9 of 10 checks passed

github-actions bot mentioned this pull request Jun 30, 2024

chore: version packages #153

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: amountToHangul이 소수점, 숫자도 대응할 수 있도록 수정 #144

fix: amountToHangul이 소수점, 숫자도 대응할 수 있도록 수정 #144

Collection50 commented Jun 29, 2024

changeset-bot bot commented Jun 29, 2024 •

edited

Loading

vercel bot commented Jun 29, 2024 •

edited

Loading

codecov-commenter commented Jun 29, 2024 •

edited

Loading

okinawaa commented Jun 29, 2024

crucifyer commented Jun 29, 2024 •

edited

Loading

okinawaa left a comment

okinawaa left a comment

fix: amountToHangul이 소수점, 숫자도 대응할 수 있도록 수정 #144

fix: amountToHangul이 소수점, 숫자도 대응할 수 있도록 수정 #144

Conversation

Collection50 commented Jun 29, 2024

Overview

AS-IS

TO-BE

PR Checklist

changeset-bot bot commented Jun 29, 2024 • edited Loading

🦋 Changeset detected

vercel bot commented Jun 29, 2024 • edited Loading

codecov-commenter commented Jun 29, 2024 • edited Loading

Codecov Report

okinawaa commented Jun 29, 2024

crucifyer commented Jun 29, 2024 • edited Loading

okinawaa left a comment

Choose a reason for hiding this comment

okinawaa left a comment

Choose a reason for hiding this comment

changeset-bot bot commented Jun 29, 2024 •

edited

Loading

vercel bot commented Jun 29, 2024 •

edited

Loading

codecov-commenter commented Jun 29, 2024 •

edited

Loading

crucifyer commented Jun 29, 2024 •

edited

Loading