Skip to content

Fix(client): FCM 로직 수정#184

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

Fix(client): FCM 로직 수정#184
jllee000 merged 21 commits intodevelopfrom
fix/#161/fcm-setting-edit

Conversation

@jllee000
Copy link
Collaborator

@jllee000 jllee000 commented Oct 3, 2025

📌 Related Issues

⭐ PR Point (To Reviewer)

[참고] 서버쪽이랑 FCM 알람 오류 실시간 확인 중이라!! 커밋 중복되게 계속 수정 중입니다 ㅠㅠ

📄 Tasks

  1. 푸시알람 클릭 시에 핀백으로 랜딩 로직
  2. url 반환

📷 Screenshot

Summary by CodeRabbit

  • 버그 수정
    • 백그라운드 푸시 알림에서 재알림 신호(renotify)와 태그(tag)를 제거했습니다. 동일 알림의 반복 진동/소리 발생을 줄이고, 태그 기반 덮어쓰기로 인한 예기치 않은 알림 교체를 방지하여 표시 동작을 보다 예측 가능하게 개선했습니다. 사용자는 중복 재알림이 감소하고, 알림이 임의로 합쳐지거나 대체되지 않는 보다 안정적인 알림 경험을 얻습니다.

@coderabbitai
Copy link

coderabbitai bot commented Oct 3, 2025

Walkthrough

FCM 서비스 워커에서 백그라운드 알림 옵션 중 renotify와 tag 속성을 제거하여 알림의 재알림 표시와 태그 식별을 사용하지 않도록 변경했습니다. 공개/내보내기 선언 변화는 없습니다.

Changes

Cohort / File(s) Summary
Firebase Messaging SW
apps/client/public/firebase-messaging-sw.js
알림 옵션에서 renotify, tag 제거. 기타 로직 및 공개 API 변경 없음.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

fix

Suggested reviewers

  • constantly-dev
  • jjangminii

Poem

토끼는 알림 숲을 살짝 손봤지요 🥕
태그도, 재알림도 조용히 놓아주고
퐁당, 하나의 종소리만 남겼답니다.
밤하늘 별처럼 깔끔한 한 번의 빛,
푸듀둥—코끝 간질, 변경 완료!

Pre-merge checks and finishing touches

❌ Failed checks (4 warnings)
Check name Status Explanation Resolution
Title Check ⚠️ Warning 제목에 이슈 번호와 슬래시가 혼용된 비표준 형식이 포함되어 있고, 실제 변경된 Firebase 메시징 작업(renotify 및 tag 옵션 제거)을 명확히 요약하지 못하고 있어 명확성과 가독성이 떨어집니다. 제목을 “Firebase 메시징 서비스 워커에서 renotify 및 tag 옵션 제거”처럼 주요 변경 내용을 간결하고 명확하게 나타내도록 수정하세요.
Linked Issues Check ⚠️ Warning 이 PR은 Firebase 알림 옵션 제거만 포함하고 있어 linked issue #25의 진행률 표시 컴포넌트 구현 요구사항과 전혀 일치하지 않습니다. PR이 연결된 issue(#25)의 요구사항을 실제로 구현하거나 올바른 이슈를 연결해 해당 기능 구현 과제를 반영하도록 수정하세요.
Out of Scope Changes Check ⚠️ Warning 변경된 firebase-messaging-sw.js의 알림 옵션 제거는 linked issue(#25)의 범위와 무관한 코드로, PR 목적에 맞지 않는 불필요한 수정입니다. linked issue의 작업 범위에 맞는 변경만 남기고 FCM 관련 수정은 제거하거나 별도 PR로 분리해 주세요.
Description Check ⚠️ Warning 설명 섹션이 템플릿의 플레이스홀더만 남긴 채 관련 이슈 번호, 작업 목록, PR 포인트 등의 필수 정보가 전혀 작성되지 않아 요구사항을 충족하지 못합니다. 관련 이슈 번호를 정확히 기재하고 수행한 작업 내용, 리뷰어에게 전달할 PR 포인트 등을 구체적으로 작성해 주세요.
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ 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

📜 Recent 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 7c2e9be and 4ab7e22.

📒 Files selected for processing (1)
  • apps/client/public/firebase-messaging-sw.js (0 hunks)
💤 Files with no reviewable changes (1)
  • apps/client/public/firebase-messaging-sw.js
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: storybook

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.

@vercel
Copy link

vercel bot commented Oct 3, 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 3, 2025 2:43am
pinback-client-landing Ready Ready Preview Comment Oct 3, 2025 2:43am

@github-actions
Copy link

github-actions bot commented Oct 3, 2025

✅ Storybook chromatic 배포 확인:
🐿️ storybook

@jllee000 jllee000 merged commit 3da8362 into develop Oct 3, 2025
11 checks passed
@jllee000 jllee000 changed the title Fix/#161/fcm setting edit Fix(client): FCM 로직 수정 Oct 11, 2025
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