Skip to content

[FIX] 아이콘 추가#117

Merged
yougyung merged 3 commits intomainfrom
fix/#97
Nov 27, 2025
Merged

[FIX] 아이콘 추가#117
yougyung merged 3 commits intomainfrom
fix/#97

Conversation

@yougyung
Copy link
Copy Markdown
Collaborator

@yougyung yougyung commented Nov 27, 2025

🔥 연관 이슈

🚀 작업 내용

  • 누락된 통계화면 Icon 추가

Summary by CodeRabbit

  • 새로운 기능

    • 정보 아이콘을 사용자 인터페이스에 추가했습니다.
  • 기타

    • 버전이 1.0.21로 업데이트되었습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@yougyung yougyung self-assigned this Nov 27, 2025
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Nov 27, 2025

Walkthrough

패키지 버전을 1.0.19에서 1.0.21로 업그레이드하고, 공유 UI 아이콘 매핑에 새로운 정보 아이콘(information.svg)을 추가했습니다. 기능적 변경 사항은 없습니다.

Changes

Cohort / File(s) 변경 요약
버전 업그레이드
package.json
버전 1.0.19에서 1.0.21로 업그레이드
아이콘 추가
src/shared/ui/assets/index.tsx
information.svg 파일을 임포트하고 Icons 매핑에 information 키 추가하여 IconName 유형에 새로운 키 포함

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

  • Icons 매핑의 새로운 엔트리 생성 및 유형 안정성 검증 필요
  • information.svg 파일이 올바르게 번들에 포함되었는지 확인

Possibly related PRs

Suggested labels

chore

Suggested reviewers

  • keemsebin
  • ujinsim

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning package.json의 버전 업데이트(1.0.19→1.0.21)는 아이콘 추가 작업과 직접적 관련이 없어 보이며, 이슈에 명시되지 않은 변경입니다. package.json 버전 업데이트가 이슈 #97과 어떤 관련이 있는지 설명하거나, 별도의 PR에서 처리해주세요.
Description check ❓ Inconclusive PR 설명은 연관 이슈와 작업 내용을 포함하고 있으나, 템플릿의 모든 필수 섹션을 완전히 채우지 않았습니다. 고민했던 내용과 리뷰 중점사항 섹션을 추가하여 더 상세한 문맥을 제공해주세요.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목 '[FIX] 아이콘 추가'는 실제 변경 사항인 정보 아이콘 추가를 적절하게 요약하고 있습니다.
Linked Issues check ✅ Passed PR은 연관 이슈 #97의 요구사항 중 '통계 아이콘 추가'를 충족하고 있으며, package.json 버전 업데이트도 포함되어 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/#97

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0e31271 and 19c4d39.

⛔ Files ignored due to path filters (1)
  • src/shared/ui/assets/information.svg is excluded by !**/*.svg
📒 Files selected for processing (2)
  • package.json (1 hunks)
  • src/shared/ui/assets/index.tsx (2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: preview-storybook
🔇 Additional comments (2)
package.json (1)

3-3: 패키지 버전 증가 적절합니다

누락된 아이콘 추가 수준의 변경이라 패치 버전(1.0.21)으로 올린 선택이 타당해 보입니다. 만약 별도 CHANGELOG를 운영 중이라면 이번 아이콘 추가 내역도 함께 기록해 두면 좋겠습니다.

src/shared/ui/assets/index.tsx (1)

16-16: 새 information 아이콘 매핑이 기존 패턴과 일관적입니다

information.svg를 임포트한 뒤 Icons 객체에 information 키로 추가해 IconName 유니온에도 자연스럽게 반영되는 구조라, 기존 아이콘 사용 방식과 완전히 일치합니다. 네이밍 컨벤션(camelCase)도 다른 항목들과 잘 맞습니다.

Also applies to: 64-64


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.

@github-actions
Copy link
Copy Markdown
Contributor

Update: 2025년 11월 27일 17시 49분 41초
Storybook 미리보기: https://677a9a76c58a295e9421d3ef-axnyztbtvg.chromatic.com/

@yougyung yougyung merged commit ac0853b into main Nov 27, 2025
9 checks passed
@yougyung yougyung deleted the fix/#97 branch November 27, 2025 08:51
@coderabbitai coderabbitai bot mentioned this pull request Feb 22, 2026
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.

[FIX] icon 추가 및 이미지업로드 스크롤 개선

1 participant