Skip to content

CHA-127: 예산 부족 채널 툴팁 금액 동적 표시 - #257

Merged
yereong merged 7 commits into
mainfrom
cha-127-fix-budget-simulation-tooltip
Aug 21, 2026
Merged

CHA-127: 예산 부족 채널 툴팁 금액 동적 표시#257
yereong merged 7 commits into
mainfrom
cha-127-fix-budget-simulation-tooltip

Conversation

@yereong

@yereong yereong commented Aug 21, 2026

Copy link
Copy Markdown
Member

무엇을 변경했나요?

  • 예산 부족 채널의 추가 필요 예산을 minBudgetWon - allocatedBudgetWon 차이로 계산해 hover 툴팁에 표시합니다.
  • isExecutable === false인 채널에 정보 확인 안내 툴팁을 표시하고, 예산 부족이지만 minBudgetWon이 없는 경우에도 동일한 안내를 제공합니다.
  • 시뮬레이션 결과 최초 표시 시 채널 툴팁을 자동 활성화하고, 2초 노출 후 1초 동안 서서히 사라지도록 처리합니다.
  • 집행 불가 채널의 텍스트와 이미지를 집행 가능 채널보다 흐리게 표시합니다.
  • 모바일 시뮬레이터 필터 사이드바의 border radius를 제거하고, 데스크톱에서는 기존 radius를 유지합니다.
  • 새로고침 후 채널 API의 iconUrl을 결과 모델에 전달하고 estimate API 이미지가 없을 때 fallback으로 사용합니다.
  • 변경된 모델·UI 테스트를 추가 및 수정했습니다.

왜 변경했나요?

예산 시뮬레이션 결과에서 채널별 집행 가능 여부와 부족 예산을 명확히 안내하고, 새로고침·모바일 환경에서도 채널 정보와 UI가 일관되게 보이도록 개선했습니다.

Linear: https://linear.app/chaesozip/issue/CHA-127/예산-시뮬레이션-툴팁and표-배지-데이터

Closes CHA-127

확인 사항

필수 확인

  • node --run build 통과 — NEXT_PUBLIC_API_BASE_URL 미설정으로 페이지 데이터 수집 단계에서 실패
  • node --run lint 통과
  • 관련 테스트 통과 — simulator-channel, AuthenticatedChannelResults 15개
  • 관련 이슈를 연결했습니다: Closes CHA-127

기능 검증

  • 변경된 기능을 로컬에서 직접 확인했습니다
  • API 응답 모델과 상태 흐름을 확인했습니다
  • 에러/로딩/빈 상태 등 예외 케이스를 확인했습니다
  • 변경된 동작에 대한 테스트를 추가하거나 업데이트했습니다

설정 및 문서

  • 문서 및 환경 변수 변경 없음
  • 새로운 의존성 추가 없음
  • 민감한 값, 토큰, 인증 정보가 포함되지 않았습니다

해당되는 경우

  • 보안 영향을 검토했습니다
  • 성능 영향 없음
  • 기존 동작 변경 내용을 PR 설명에 작성했습니다
  • 스크린샷 또는 화면 녹화 첨부

@linear-code

linear-code Bot commented Aug 21, 2026

Copy link
Copy Markdown

CHA-127

@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: a4baf0f9-7e75-4643-9b44-efe2b18efc00


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.

@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.00000% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/pages/simulator/model/simulator-channel.ts 88.46% 1 Missing and 2 partials ⚠️
...ges/simulator/ui/simulator-channel-performance.tsx 90.00% 0 Missing and 2 partials ⚠️

📢 Thoughts on this report? Let us know!

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions github-actions Bot added size/L and removed size/M labels Aug 21, 2026
@yereong
yereong marked this pull request as ready for review August 21, 2026 04:41
@github-actions
github-actions Bot requested a review from DongjaJ August 21, 2026 04:41
@yereong
yereong merged commit 9fbd6da into main Aug 21, 2026
7 checks passed
@yereong
yereong deleted the cha-127-fix-budget-simulation-tooltip branch August 21, 2026 04:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants