Skip to content

fix: 현재 적용중인 정책 GB 변환 통일#52

Merged
haeni82 merged 1 commit into
developfrom
feature/policy-api
Mar 16, 2026
Merged

fix: 현재 적용중인 정책 GB 변환 통일#52
haeni82 merged 1 commit into
developfrom
feature/policy-api

Conversation

@haeni82
Copy link
Copy Markdown
Contributor

@haeni82 haeni82 commented Mar 16, 2026

이슈

  • closed #이슈번호

✔️ 체크리스트

  • : Merge할 브랜치를 확인해 주세요.

🔍 작업 내용

  • 현재 적용중인 정책 GB 변환 유틸함수 통일

⚠️ 주의 사항 / 기타

Summary by CodeRabbit

릴리스 노트

  • 리팩토링
    • 내부 코드 개선 사항이 적용되었습니다. 사용자에게 직접적인 영향을 주는 변경사항은 없습니다.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 16, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

useAppliedPolicies.ts 훅에서 수동 GB 계산 로직을 formatData 유틸리티 함수로 교체했습니다. 임포트 추가 및 배열 표현식 포맷팅 조정이 수반되었으며, 훅의 공개 인터페이스와 정책 집계 동작은 변경되지 않았습니다.

Changes

Cohort / File(s) Summary
데이터 포맷팅 유틸리티 적용
src/page/PolicyDetail/hooks/useAppliedPolicies.ts
formatData 유틸리티 함수 임포트 추가, 하드코딩된 GB 변환 로직 제거 및 formatData(dailyDataLimit), formatData(sharedDataLimit) 호출로 교체, 배열 필터/정렬 표현식 포맷팅 개선

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 변경 사항의 핵심을 명확하게 설명하고 있으며, GB 변환 유틸함수 통일이라는 주요 변경 사항을 잘 반영하고 있습니다.
Description check ✅ Passed PR 설명이 필수 섹션을 포함하고 있으나, 이슈 번호가 채워지지 않았고 체크리스트 항목이 완료되지 않았습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/policy-api
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@haeni82 haeni82 merged commit bbe6fa2 into develop Mar 16, 2026
1 of 2 checks passed
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.

1 participant