Skip to content

fix : 포어그라운드 트래킹 알림 시스템 배너 활성화#126

Merged
JioCoder merged 1 commit into
mainfrom
fix/tracking-fcm-notification
Jun 4, 2026
Merged

fix : 포어그라운드 트래킹 알림 시스템 배너 활성화#126
JioCoder merged 1 commit into
mainfrom
fix/tracking-fcm-notification

Conversation

@JioCoder

@JioCoder JioCoder commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

요약

  • FCM silent push → mixed payload(notification + data) 전환 대응
  • 트래킹 푸시 알림 중복 제거 및 포어그라운드/백그라운드 시스템 배너 정상화
  • 데모 모드 카메라 동작 개선 (출발 좌표 고정, follow 비활성화)
  • 코스 폴리라인 두께 7/11px 고정

변경 파일

  • index.js — 백그라운드 핸들러에서 트래킹 타입 로컬 알림 생성 제거 (mixed payload로 OS 자동 표시)
  • features/tracking/hooks/use-tracking-fcm.ts — 포어그라운드 scheduleNotificationAsync 중복 제거
  • app/_layout.tsxsetNotificationHandler 타입별 배너 분기 추가, 트래킹 타입 포어그라운드 시스템 배너 활성화
  • app/(tabs)/tracking.tsx — 데모 모드 카메라 follow 비활성화 / 트래킹 시작 시 출발 좌표 줌 / 폴리라인 두께 고정

테스트 체크리스트

  • 포어그라운드 트래킹 중 마일스톤 도달 시 시스템 배너 + in-app 배너 둘 다 표시 확인
  • 백그라운드에서 마일스톤 도달 시 시스템 푸시 알림 정상 수신 확인
  • 커뮤니티 등 다른 알림 시스템 배너 정상 표시 확인
  • 데모 모드 트래킹 시작 시 카메라가 출발 좌표로 이동 확인
  • 코스 폴리라인 두께 정상 표시 확인

연관 백엔드 PR

  • SEMOSAN_BE #190 — 트래킹 푸시 mixed payload 전환

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the notification handler in app/_layout.tsx to enable banners, lists, and sounds for TRACKING_PHOTO_MILESTONE and TRACKING_SUMMIT_REACHED notification types. The reviewer pointed out that this change makes the conditional block redundant as it now matches the default return behavior, and suggested simplifying the code by removing the unnecessary condition and updating the outdated comments.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread app/_layout.tsx
@JioCoder JioCoder merged commit 3b61d5d into main Jun 4, 2026
2 checks passed
@JioCoder JioCoder deleted the fix/tracking-fcm-notification branch June 4, 2026 15:13
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.

2 participants