Skip to content

Conversation

@jllee000
Copy link
Collaborator

@jllee000 jllee000 commented Jan 18, 2026

📌 Related Issues

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

📄 Tasks

⭐ PR Point (To Reviewer)

공통 도메인 www. 제거합니다

📷 Screenshot

Summary by CodeRabbit

릴리스 노트

  • 버그 수정

    • 플랫폼 전체에서 도메인 URL 표준화 (www 제거)
    • 인증 오류 시 토큰 정리 로직 조정
  • 최적화

    • 이미지 자산 형식을 WEBP에서 SVG로 변경

✏️ 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 8:06am
pinback-client-landing Ready Ready Preview, Comment Jan 18, 2026 8:06am

@jllee000 jllee000 self-assigned this Jan 18, 2026
@github-actions github-actions bot added the fix 버그 수정하라 러브버그 label Jan 18, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 18, 2026

Walkthrough

여러 파일의 Pinback URL에서 'www' 서브도메인을 제거하는 도메인 정규화 작업, MacStep 컴포넌트의 자산 import를 WEBP에서 SVG로 변경, 그리고 401/403 에러 처리 시 토큰 제거 로직을 비활성화하는 변경 사항이 포함됨.

Changes

코호트 / 파일 변경 요약
도메인 URL 정규화
apps/client/public/firebase-messaging-sw.js, apps/extension/src/App.tsx, apps/extension/src/background.ts, apps/extension/src/pages/LogOutPop.tsx, apps/extension/src/shared/components/Header.tsx
기본 Pinback URL에서 www 서브도메인 제거 (https://www.pinback.todayhttps://pinback.today)
자산 import 변경
apps/client/src/pages/onBoarding/components/funnel/step/MacStep.tsx
MacNotice 자산을 WEBP 이미지에서 SVG 이미지로 변경 (사용 방식 동일)
에러 처리 로직 수정
apps/client/src/shared/apis/setting/axiosInstance.ts
401/403 응답 에러 처리 시 localStorage.removeItem('token') 제거 로직을 주석 처리 (리다이렉션은 유지)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

fix, frontend

Suggested reviewers

  • constantly-dev
  • jjangminii
  • karnelll

Poem

🐰 www를 벗고 핑백이 가벼워졌네,
SVG로 갈아입은 Mac도 멋졌고,
토큰 정리는 뒤로하고—
도메인 단정, 자산 새로움, 로직 다듬기,
작은 손질이 모여 서비스를 다시 쓴다! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 3
❌ Failed checks (1 warning, 2 inconclusive)
Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning 설명된 변경 사항 중 localStorage 토큰 삭제 로직 제거(axiosInstance.ts)는 #243의 도메인 수정 범위 외의 변경으로 보여, 명시된 목표와의 연관성이 불명확합니다. localStorage.removeItem('token') 제거가 #243의 도메인 수정과 어떤 관련이 있는지, 또는 별도의 QA 항목인지 명확히 해주세요.
Title check ❓ Inconclusive PR 제목은 "도메인 클라쪽 수정"으로 실제 변경 사항(www 도메인 제거)을 언급하지만 구체성이 부족하고 문법적 오류가 있어 다소 모호합니다. PR 제목을 "fix: www 도메인 제거" 또는 "fix: pinback.today 도메인 정규화"로 수정하여 더 명확하게 변경 사항을 표현해주세요.
Linked Issues check ❓ Inconclusive PR은 #243 QA 대응 항목을 해결하고 있으나, 변경 사항 일부(토큰 삭제 로직 제거, 이미지 변경)가 #243과의 명확한 연계가 불명확하며, #25 Progress 컴포넌트 구현과는 무관합니다. 각 변경 사항이 #243의 어느 QA 항목을 해결하는지, 그리고 토큰 삭제 로직 제거의 의도가 무엇인지 명확히 설명해주세요.
✅ Passed checks (2 passed)
Check name Status Explanation
Description check ✅ Passed PR 설명은 관련 이슈(#243)를 태그하고 작업 내용(공통 도메인 www 제거)을 명확히 명시했으나, Tasks 섹션이 비어있어 구체적인 작업 목록이 부족합니다.
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.

@github-actions
Copy link

✅ Storybook chromatic 배포 확인:
🐿️ storybook

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.

jinman

@jllee000 jllee000 merged commit 20c01c7 into develop Jan 18, 2026
11 checks passed
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
apps/client/src/shared/apis/setting/axiosInstance.ts (1)

40-52: 401/403 응답 시 토큰을 제거하지 않아 인증 상태 불일치 및 반복적인 요청 실패 발생 가능

request 인터셉터는 localStorage에 토큰이 존재하는 것만으로 Authorization 헤더를 자동으로 붙입니다(라인 12-20). 하지만 401/403 응답 핸들러에서 토큰을 제거하지 않으면(라인 49 주석 처리됨), 만료되거나 폐기된 토큰이 localStorage에 남아있게 되고, 사용자가 리다이렉트된 후 다른 API 호출을 수행할 때도 이 무효한 토큰이 계속 전송됩니다. 이는 다른 엔드포인트에서 반복적인 401 실패나 UI상 로그인 상태 불일치를 초래할 수 있습니다. 토큰을 제거하여 이후 요청이 토큰 없이 진행되도록 하거나, 명시적으로 Authorization 헤더 전송을 차단해야 합니다.

🔧 제안 수정
-      // localStorage.removeItem('token');
+      localStorage.removeItem('token');

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix 버그 수정하라 러브버그

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fix] 2nd Sprint - QA

3 participants