Skip to content

CHA-158 채널 상세 광고 예시 이미지 갤러리 - #262

Merged
DongjaJ merged 3 commits into
mainfrom
cha-158-feat-channel-detail-preview-gallery
Aug 21, 2026
Merged

CHA-158 채널 상세 광고 예시 이미지 갤러리#262
DongjaJ merged 3 commits into
mainfrom
cha-158-feat-channel-detail-preview-gallery

Conversation

@DongjaJ

@DongjaJ DongjaJ commented Aug 21, 2026

Copy link
Copy Markdown
Member

무엇을 변경했나요?

  • 채널 상세 API의 previewImageUrls를 화면 모델에 연결하고 URL의 공백·빈 값·중복을 정규화했습니다.
  • URL만 제공되는 이미지를 next/image 기반 Pinterest형 multi-column 갤러리로 표시합니다.
  • 이미지가 없으면 기존 집행 사례를, 둘 다 없으면 빈 상태를 표시합니다.
  • 혼합 이미지·단일 이미지·집행 사례·빈 상태 Storybook과 관련 테스트를 추가했습니다.
  • 채널 상세 모달 닫기 버튼에 pointer cursor를 적용했습니다.

왜 변경했나요?

채널 상세 모달에서 실제 광고 집행 이미지를 원본 비율에 맞춰 한눈에 확인할 수 있도록 하기 위해 변경했습니다.

Linear: https://linear.app/chaesozip/issue/CHA-158/feature-채널-상세-광고-예시-이미지-갤러리

Closes CHA-158

확인 사항

필수 확인

  • node --run build 통과
  • node --run lint 통과
  • node --run test 통과 또는 관련 테스트 확인
  • 관련 이슈를 연결했습니다. 예: Closes #123

기능 검증

  • 변경된 기능을 로컬에서 직접 확인했습니다
  • 사용자 플로우가 의도대로 동작하는지 확인했습니다
  • API 연동 또는 상태 관리 변경 사항을 확인했습니다
  • 에러/로딩/빈 상태 등 예외 케이스를 확인했습니다
  • 변경된 동작에 대한 테스트를 추가하거나 업데이트했습니다

설정 및 문서

  • 사용자에게 노출되는 문서를 업데이트했습니다. 예: README.md, 사용 가이드 등
  • 새로운 의존성이나 설정 변경 사항을 문서화했습니다
  • 환경 변수 추가/변경 사항을 명확히 설명했습니다
  • 민감한 값, 토큰, 인증 정보가 포함되지 않았습니다

해당되는 경우

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

@linear-code

linear-code Bot commented Aug 21, 2026

Copy link
Copy Markdown

CHA-158

@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: 62199395-ba56-4991-90bb-37a9c7e39ca2


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 65.38462% with 9 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...annel-detail/ui/panels/channel-preview-gallery.tsx 55.00% 6 Missing and 3 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.

@DongjaJ
DongjaJ marked this pull request as ready for review August 21, 2026 07:21
@github-actions
github-actions Bot requested a review from yereong August 21, 2026 07:21
@DongjaJ
DongjaJ merged commit 96a2c9d into main Aug 21, 2026
9 checks passed
@DongjaJ
DongjaJ deleted the cha-158-feat-channel-detail-preview-gallery branch August 21, 2026 14:17
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