Skip to content

[KW-553] fix: fcm 토큰 없을때 null error 나지 않도록 수정#22

Merged
coffeesigma merged 1 commit intodevelopfrom
KW-553/feat/member-grpc-fcm-token-nullable
May 31, 2025
Merged

[KW-553] fix: fcm 토큰 없을때 null error 나지 않도록 수정#22
coffeesigma merged 1 commit intodevelopfrom
KW-553/feat/member-grpc-fcm-token-nullable

Conversation

@coffeesigma
Copy link
Contributor

@coffeesigma coffeesigma commented May 31, 2025

🔷 Jira Ticket ID

KW-553


📌 작업 내용 및 특이사항

  • null error 수정

📚 참고사항

@coffeesigma coffeesigma requested review from Copilot and midday2612 May 31, 2025 02:16
@github-actions github-actions bot changed the title fix: fcm 토큰 없을때 null error 나지 않도록 수정 [KW-553] fix: fcm 토큰 없을때 null error 나지 않도록 수정 May 31, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes a null error issue when the FCM token is absent by using Optional to safely default to an empty string. Key changes include:

  • Importing java.util.Optional.
  • Updating the FCM token assignment to avoid null values.
  • Ensuring the gRPC response is built without null errors.
Comments suppressed due to low confidence (1)

src/main/java/com/doubleo/memberservice/domain/member/grpc/server/MemberGrpcServiceImpl.java:33

  • While using Optional to default to an empty string resolves the null error, please verify that an empty string is the appropriate default for downstream processing to avoid unintended behavior.
Optional.ofNullable(res.getFcmToken())

Copy link
Contributor

@midday2612 midday2612 left a comment

Choose a reason for hiding this comment

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

굿

@coffeesigma coffeesigma merged commit 7a3592a into develop May 31, 2025
3 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.

3 participants