Skip to content

Conversation

@jllee000
Copy link
Collaborator

@jllee000 jllee000 commented Jan 18, 2026

📌 Related Issues

관련된 Issue를 태그해주세요. (e.g. - close #25)

📄 Tasks

  1. 맥북 설명 이미지 svg 포맷 롤백
  2. 로그인 토큰삭제 로직 제거

⭐ PR Point (To Reviewer)

📷 Screenshot

Summary by CodeRabbit

변경 사항

  • 최적화

    • 온보딩 단계의 이미지 자산 형식을 개선하였습니다.
  • 개선

    • 인증 오류 처리 흐름이 조정되었습니다.

본 릴리스에는 사용자 경험 개선을 위한 내부 최적화가 포함되어 있습니다. 이미지 형식 개선으로 더욱 효율적인 로딩 성능을 제공하며, 인증 처리 로직이 개선되어 더욱 안정적인 서비스를 제공합니다.

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

@vercel
Copy link

vercel bot commented Jan 18, 2026

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

Project Deployment Review Updated (UTC)
pinback-client-client Ready Ready Preview, Comment Jan 18, 2026 7:12am
pinback-client-landing Ready Ready Preview, Comment Jan 18, 2026 7:12am

@jllee000 jllee000 self-assigned this Jan 18, 2026
@jllee000 jllee000 requested a review from jjangminii January 18, 2026 07:11
@github-actions github-actions bot added the fix 버그 수정하라 러브버그 label Jan 18, 2026
@jllee000 jllee000 removed the fix 버그 수정하라 러브버그 label Jan 18, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 18, 2026

Walkthrough

MacStep 컴포넌트의 MacNotice 자산 import를 WebP에서 SVG로 변경하고, 인증 오류(401/403) 발생 시 localStorage에서 토큰 제거 로직을 비활성화했습니다.

Changes

Cohort / File(s) 변경 요약
자산 타입 변경
apps/client/src/pages/onBoarding/components/funnel/step/MacStep.tsx
MacNotice 이미지 import를 WebP 래스터 형식에서 SVG 벡터 형식으로 변경
인증 오류 처리
apps/client/src/shared/apis/setting/axiosInstance.ts
401/403 응답 시 localStorage에서의 토큰 제거 로직을 주석 처리하여 비활성화

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • PR #107: MacStep.tsx 파일 수정—동일한 온보딩 컴포넌트 내에서 heading 텍스트 변경
  • PR #235: axiosInstance.ts의 401/403 오류 핸들링 수정—redirect URL 변경 작업
  • PR #236: axiosInstance.ts의 인증 오류 핸들링 변경—redirect 경로 수정

Suggested labels

fix

Suggested reviewers

  • constantly-dev
  • jjangminii

Poem

🐰 SVG로 변신한 맥 아이콘,
토큰의 슬픈 작별은 미루고,
조용한 고침이 스며드네,
작지만 정확한 손길로요! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목은 두 가지 변경 사항을 명확하게 요약합니다: 이미지 변경 및 토큰 삭제 로직 제거, 이는 실제 코드 변경과 일치합니다.
Description check ✅ Passed PR 설명은 템플릿 구조를 따르고 있으며 관련 이슈, 작업 내용을 명확히 기재했으나 PR Point는 비워져 있습니다.
Linked Issues check ✅ Passed PR은 이슈 #243의 2차 스프린트 QA 항목을 다루고 있으나, 링크된 이슈 #25(Progress Bar 구현)와는 무관합니다. 코드 변경이 이슈 요구사항을 충족합니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 이슈 #243 범위 내에 있습니다: 맥북 이미지 SVG 포맷 롤백 및 토큰 삭제 로직 제거는 QA 수정사항으로 적절합니다.
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

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.

Copy link
Collaborator

@jjangminii jjangminii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙏🙏🙏🙏🙏🙏🙏🙏 진맨 plz

@github-actions
Copy link

✅ Storybook chromatic 배포 확인:
🐿️ storybook

@jllee000 jllee000 merged commit 6d7e1a1 into develop Jan 18, 2026
11 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.

[Fix] 2nd Sprint - QA

3 participants