Skip to content

Comments

Fix(client): FCM 알람 커스텀 수정#172

Merged
jllee000 merged 9 commits intodevelopfrom
fix/#161/fcm-setting-edit
Oct 1, 2025
Merged

Fix(client): FCM 알람 커스텀 수정#172
jllee000 merged 9 commits intodevelopfrom
fix/#161/fcm-setting-edit

Conversation

@jllee000
Copy link
Collaborator

@jllee000 jllee000 commented Oct 1, 2025

📌 Related Issues

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

📄 Tasks

  1. 알람 썸네일 및 텍스트 수정
  2. 익스텐션 썸네일

⭐ PR Point (To Reviewer)

📷 Screenshot

Summary by CodeRabbit

  • 새 기능
    • 배경 푸시 알림의 기본 콘텐츠가 업데이트되었습니다.
      • 기본 제목: ‘pinback’으로 변경
      • 기본 메시지: ‘저장한 북마크를 확인해 보세요!’로 변경
      • 기본 아이콘: ‘/FCM-IMG.png’로 변경
    • 앱을 사용하지 않을 때 도착하는 알림의 가독성과 인지가 향상되었습니다.

@jllee000 jllee000 self-assigned this Oct 1, 2025
@vercel
Copy link

vercel bot commented Oct 1, 2025

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

Project Deployment Preview Comments Updated (UTC)
pinback-client-client Ready Ready Preview Comment Oct 1, 2025 3:40pm
pinback-client-landing Ready Ready Preview Comment Oct 1, 2025 3:40pm

@jllee000 jllee000 added the fix 버그 수정하라 러브버그 label Oct 1, 2025
@coderabbitai
Copy link

coderabbitai bot commented Oct 1, 2025

Walkthrough

FCM 서비스 워커의 백그라운드 알림 기본 제목/본문/아이콘 문자열이 변경되었고, CardEditModal 내 getKSTISOString 함수가 동일한 이름으로 중복 선언되었습니다. 제어 흐름 변화는 없으며, 서비스 워커는 내용만 변경되었습니다.

Changes

Cohort / File(s) Summary
FCM Service Worker
apps/client/public/firebase-messaging-sw.js
백그라운드 알림 기본값 변경: 제목 'PINBACK'→'pinback', 본문 '알림이 도착했어요!'→'저장한 북마크를 확인해 보세요!', 아이콘 '/link_Thumbnail.png'→'/FCM-IMG.png'. 로직 변화 없음.
CardEditModal Date Utils
apps/client/src/shared/components/cardEditModal/CardEditModal.tsx
getKSTISOString가 동일 스코프에 중복 선언됨. 재선언 충돌로 컴파일 에러 가능성. 기타 로직 변화 없음.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

Poem

새 알림 제목, 살짝 낮춰 핀백-핀백,
아이콘도 번쩍, 북마크로 쾅쾅! ⭐
하지만 함수는 둘이면 너무 많죠…
한 줄 지우고, 빌드도 살리고,
깡총깡총 PR 위에 춤춘다, 토끼 🐇

Pre-merge checks and finishing touches

❌ Failed checks (3 warnings)
Check name Status Explanation Resolution
Linked Issues Check ⚠️ Warning PR 변경이 이슈 #161의 FCM 알림 수정 요구사항만 반영하고 이슈 #25의 프로그래스바 구현 요구사항은 전혀 포함하지 않아 연결된 이슈 전체의 목표를 충족하지 못했습니다. PR에서 이슈 #25를 제거하거나 해당 이슈의 프로그래스바 구현을 추가로 수행해야 합니다.
Out of Scope Changes Check ⚠️ Warning CardEditModal.tsx 파일에 동일한 getKSTISOString 함수가 중복 선언되어 있으며 이는 연결된 이슈의 목표와 무관한 변경사항으로 보입니다. 중복 선언된 getKSTISOString 함수를 제거하여 변경 범위를 이슈 목표에 맞게 정리해야 합니다.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed 제목 “Fix(client): FCM 알람 커스텀 수정”은 클라이언트 측 FCM 알림의 기본 텍스트·썸네일 변경이라는 핵심 변경사항을 명확히 요약하고 있어 PR의 주요 목적과 일치합니다.
Description Check ✅ Passed 제공된 설명은 관련 이슈(close #161)와 주요 작업 항목(알람 썸네일 및 텍스트 수정, 익스텐션 썸네일)을 포함해 템플릿의 핵심 섹션을 충족하고 있어 PR 이해에 충분합니다.
✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/#161/fcm-setting-edit

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

github-actions bot commented Oct 1, 2025

✅ Storybook chromatic 배포 확인:
🐿️ storybook

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: 1

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 78cd389 and 9a6854b.

⛔ Files ignored due to path filters (2)
  • apps/client/public/FCM-IMG.png is excluded by !**/*.png
  • apps/extension/public/icon.png is excluded by !**/*.png
📒 Files selected for processing (2)
  • apps/client/public/firebase-messaging-sw.js (1 hunks)
  • apps/client/src/shared/components/cardEditModal/CardEditModal.tsx (1 hunks)
🧰 Additional context used
🪛 GitHub Actions: ci
apps/client/src/shared/components/cardEditModal/CardEditModal.tsx

[error] 80-80: Build failed during 'tsc -b && vite build'. TypeScript error TS2393: Duplicate function implementation.

🔇 Additional comments (1)
apps/client/public/firebase-messaging-sw.js (1)

36-39: FCM 알림 기본값 변경 및 아이콘 파일 확인 완료
알림 제목, 본문, 아이콘 기본값이 의도대로 변경되었으며, /apps/client/public/FCM-IMG.png 파일이 존재함을 확인했습니다.

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] FCM 커스텀 및 연동 수정

1 participant