[Deploy] - 솝탬프 어드민 알림 기능 배포 #649
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 솝탬프 딥링크에서 처리하는 방식은 고정이므로 해당 빌더를 호출하는 쪽에서 알고 있는 것이 좋지 않다고 판단함
- 검증 로직을 내부에 위치시킴. 빌더패턴에서 영감을 받은 리턴값을 전달해주는 validate 메서드 사용 - builderWithRequired 를 통해 필수값 누락을 방지
- 전달할 userId가 비었을 경우, 전달하지 않도록 방어 로직 구현 - InstantlyAlarmSender 에선 InstantAlarmRequest를 파라미터로 강제하도록 typeArgument 지정
- 주석에 명시한대로 현재는 앱의 변경 사항을 적용하기 애매한 부분이 있어서 알림을 두 방향으로 보냄. 1.작성자 2.그 외. 추후 클라이언트 측에서 isMine 값 판단 여부를 스탬프 상세 조회에서 하도록 반영되면 deepLink에서 isMine 부분을 제외하고 메서드를 정리해야 함 - 현재는 admin 으로 생성하는 알림만 instantAlarmSender 를 사용하지만, 추후 clap이나 poke 에서도 해당 AlarmSender를 바라보도록 조정하는 것이 좋음
- 아무나 접근할 수 없도록 서버의 별도 api 키를 사용하도록 구현했기 때문.
- 클라 측에서 앰플리튜드로 통계 정보를 기록 시 soptamp 상세 조회에서 ownerNickName을 알아야해서 추가함
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related issue 🛠
Work Description ✏️
Trouble Shooting ⚽️
Related ScreenShot 📷
Uncompleted Tasks 😅
To Reviewers 📢