Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[DPMBE-126] 약속 생성 시 동작하는 트래킹 핸들러 키 불일치문제와 fcm send 버그 해결 #213

Merged
merged 9 commits into from
Jul 19, 2023
Merged

[DPMBE-126] 약속 생성 시 동작하는 트래킹 핸들러 키 불일치문제와 fcm send 버그 해결 #213

merged 9 commits into from
Jul 19, 2023

Conversation

kdomo
Copy link
Member

@kdomo kdomo commented Jul 18, 2023

개요

작업사항

  • Redis Repository로 값을 저장하게되어 key값에 prefix가 붙게되면서 이벤트가 정상적으로 발행되지 않아 prefix제거하였습니다.
  • 앱 알림 허용한 유저 리스트를 만들어두고 해당 리스트로 fcm message를 전송하는데, 리스트가 비어있을때 처리되지 않아 조건 추가하였습니다
  • 트래킹종료 이벤트EXPIRE_EVENT_TRACKING_TIME_START_{promiseId} 네이밍이 START로 되어있어,
    EXPIRE_EVENT_TRACKING_TIME_END_{promiseId}로 변경 (START -> END)

@kdomo kdomo self-assigned this Jul 18, 2023
ImNM
ImNM approved these changes Jul 18, 2023
View reviewed changes
Copy link
Member

@ImNM ImNM left a comment

Choose a reason for hiding this comment

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

볼... 시간이없었네요 지송

@kdomo kdomo mentioned this pull request Jul 19, 2023
Merged
@kdomo
feat: 인터렉션 발송 시 앱 알람이 꺼져있을때 푸시알림만 안보내도록 한다. (#207)
a57e4d9
@kdomo
[DPMBE-123] 트래킹 시작 시 해당 약속에 참여한 사람이 없다면 트래킹을 시작하지 않는다. (#208)
8aab7b7 
* feat: 트래킹 시작 시 해당 약속에 참여한 사람이 없다면 트래킹을 시작하지 않는다.

* feat: 트래킹 시작 시 약속 유저가 2명 이하라면 약속을 종료 처리시킨다

* fix: 약속 참여유저 2 미만 이벤트 삭제 시 람다식으로 변경
@kdomo
[DPMBE-116] 자신의 알림을 조회한다 (#211)
1a8d160 
* fix: 인터렉션 타입에 Enumerated String을 사용하도록 변경

* feat: 자신의 알림을 조회한다
@kdomo
fix: Default Argument에서 Named Argument로 변경
8b91654
@kdomo
Merge branch 'develop' into fix/DPMBE-126
ce867af
@kdomo
style: 주석제거
abe8f4d
@sonarcloud
Copy link

sonarcloud bot commented Jul 19, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

BlackBean99
BlackBean99 approved these changes Jul 19, 2023
View reviewed changes
Copy link
Collaborator

@BlackBean99 BlackBean99 left a comment

Choose a reason for hiding this comment

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

LGTM

@kdomo kdomo merged commit 8e19c78 into develop Jul 19, 2023
3 checks passed
@kdomo kdomo deleted the fix/DPMBE-126 branch July 19, 2023 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ImNM ImNM ImNM approved these changes

@BlackBean99 BlackBean99 BlackBean99 approved these changes

Assignees

@kdomo kdomo

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[DPMBE-126] 약속 생성 시 동작하는 트래킹 핸들러 키 불일치문제와 fcm send 버그 해결
3 participants
@kdomo @ImNM @BlackBean99