Skip to content

[Fix] 튜토리얼 카드뉴스에서 썸네일 이미지 제거 - #233

Merged
whghdus merged 2 commits into
devfrom
fix/232-remove-tutorial-news-image
Sep 19, 2026
Merged

whghdus merged 2 commits into
devfrom
fix/232-remove-tutorial-news-image

Conversation

@whghdus

@whghdus whghdus commented Sep 19, 2026 •

Copy link
Copy Markdown
Contributor

📌 관련 이슈

🏷️ PR 타입

  • 🎨 스타일/마크업 변경 (Style)

🖥 관련 화면

  • SCR-03: 튜토리얼 STEP 1 (카드뉴스 상세)

📝 작업 내용

  • 튜토리얼 목업 데이터(TUTORIAL_NEWS_CARD_MOCK, TUTORIAL_NEWS_CARD_MOCK_2)에서 imageUrl 제거 → NewsCardInfo의 이미지 블록이 조건부 렌더링이라 자동으로 숨겨짐
  • 더 이상 참조되지 않는 목업 썸네일 에셋(newscardImg.svg, newscardImg2.svg, 합쳐서 약 5MB) 삭제

📸 스크린샷 (UI 변경 시 필수)

Before After
카드뉴스 상세 상단에 썸네일 이미지 노출 배지/날짜/헤드라인 다음 바로 3줄 요약

✅ 체크리스트

공통

  • 셀프 리뷰를 완료했습니다
  • pnpm lint / pnpm typecheck 를 통과했습니다
  • 브랜치명 컨벤션을 준수했습니다 (fix/232-remove-tutorial-news-image)

UI / 스타일 변경 시

  • 모바일 우선으로 확인했고, 데스크탑에서도 깨지지 않습니다

🤖 Generated with Claude Code

Summary by CodeRabbit

  • 정리
    • 사용하지 않던 튜토리얼 뉴스 카드 이미지 참조를 제거했습니다.
    • 튜토리얼 뉴스 카드 예시 데이터에서 이미지 정보가 삭제되었습니다.

- TUTORIAL_NEWS_CARD_MOCK / MOCK_2에서 imageUrl 제거
- 더 이상 쓰이지 않는 목업 썸네일 에셋(newscardImg.svg, newscardImg2.svg) 삭제

Closes #232

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@vercel

vercel Bot commented Sep 19, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
brifo-front Ready Ready Preview Sep 19, 2026 10:26am UTC

@coderabbitai

coderabbitai Bot commented Sep 19, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (2)
  • src/assets/images/newscardImg.svg is excluded by !**/*.svg
  • src/assets/images/newscardImg2.svg is excluded by !**/*.svg

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Repository: Team-BRIFO/BRIFO-front/.coderabbit.yaml

Review profile: CHILL

Plan: Essentials

Run ID: 6596b4a5-883e-4304-b70a-d37fb61d58b6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: Team-BRIFO/BRIFO-front/.coderabbit.yaml

Review profile: CHILL

Plan: Essentials

Run ID: 62eec70a-c32e-48f2-ad77-84467a81ff86

📥 Commits

Reviewing files that changed from the base of the PR and between 8aa02aa and 673db1e.

⛔ Files ignored due to path filters (2)
  • src/assets/images/newscardImg.svg is excluded by !**/*.svg
  • src/assets/images/newscardImg2.svg is excluded by !**/*.svg
📒 Files selected for processing (1)
  • src/pages/TutorialPage/mockData.ts
💤 Files with no reviewable changes (1)
  • src/pages/TutorialPage/mockData.ts

Included review availability: 3 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


Walkthrough

튜토리얼 뉴스 카드 mock 데이터에서 썸네일 이미지 import와 imageUrl 필드를 제거했습니다. 두 카드 데이터 모두 썸네일 없이 구성됩니다.

Changes

튜토리얼 카드뉴스 썸네일 제거

Layer / File(s) Summary
썸네일 참조 및 import 제거
src/pages/TutorialPage/mockData.ts
사용하지 않는 NewscardImg, NewscardImg2 import를 제거했습니다. TUTORIAL_NEWS_CARD_MOCK와 TUTORIAL_NEWS_CARD_MOCK_2에서 imageUrl 필드를 제거했습니다.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix · Severity of issue fixed: Low

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Linked Issues check ❓ Inconclusive 이슈 #232의 TUTORIAL_NEWS_CARD_MOCK 및 TUTORIAL_NEWS_CARD_MOCK_2에서 imageUrl 제거는 변경 요약으로 확인됩니다. 그러나 src/assets/images/newscardImg.svg와 src/assets/images/newscardImg2.svg는 검토 제외 파일입니다. 따라서 두 에셋의 삭… 검토 가능한 diff 또는 독립적인 증거로 두 SVG 에셋의 삭제 여부를 확인해야 합니다.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 튜토리얼 카드뉴스에서 썸네일 이미지를 제거하는 주요 변경 사항을 정확하고 간결하게 설명합니다.
Out of Scope Changes check ✅ Passed 확인 가능한 변경은 src/pages/TutorialPage/mockData.ts의 사용하지 않는 이미지 import 및 튜토리얼 목업 데이터의 imageUrl 제거입니다. 변경은 이슈 #232의 썸네일 제거 범위와 직접 연결됩니다. 범위를 벗어난 변경은 확인되지 않습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Full details: Linked Issues check

Explanation

이슈 #232의 TUTORIAL_NEWS_CARD_MOCK 및 TUTORIAL_NEWS_CARD_MOCK_2에서 imageUrl 제거는 변경 요약으로 확인됩니다. 그러나 src/assets/images/newscardImg.svg와 src/assets/images/newscardImg2.svg는 검토 제외 파일입니다. 따라서 두 에셋의 삭제 여부는 확인할 수 없습니다.

✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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

@whghdus
whghdus merged commit 81e5a93 into dev Sep 19, 2026
4 checks passed
@whghdus
whghdus deleted the fix/232-remove-tutorial-news-image branch September 19, 2026 10:29

This branch was successfully deployed

1 active deployment
Preview — 979d1151 Deployed Sep 19, 2026 by vercel[bot]
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] 튜토리얼 카드뉴스에서 썸네일 이미지 제거

1 participant