Feature/#341 분봉 리팩토링 및 장 시간대에 분봉 받아오기, alarm 한번만 작동 #343
Conversation
There was a problem hiding this comment.
현재 유저가 로그아웃을 했을 때에도 주춤 사이트에 접속해 있는 상태이고, 같은 디바이스를 사용한다면 무조건 알림이 가는 상태입니다
이걸 막을 수 있는 방법이 없을까요?
알림을 한번만 보내게 설정을 바꿨는 데, 이게 맞는 지 궁금합니다! 알림 확인 버튼을 눌렀을 때 더이상 안 보내고, 취소 버튼 누르면 다시 보내게도 만들 수 있긴 한데... 뭔가 이상해서요.
취소버튼을 만들 때 다시 보내지는게 이상하긴하네요. 제 생각에는 계속 보관하는게 좋다고 생각하지만, 시간이 오래걸린다면 확인 버튼을 누르면 더 이상 전달하지 않고 그렇지 않으면 나중에 다시 보낼 수 있도록 하는건 어떤가요?
| .groupBy('stock.id') | ||
| .orderBy('alarmCount', 'DESC') | ||
| .limit(this.STOCK_LIMITS) | ||
| .execute(); |
There was a problem hiding this comment.
알람 카운터가 많은 순으로 보내긴 하지만, 저희가 구현한 우선순위큐는 마지막에 실행될거에요
| this.logger.info(`${stockId} has invalid minute data`); | ||
| return; | ||
| } | ||
| const minuteDatas: MinuteData[] = output2.map((val): MinuteData => { |
There was a problem hiding this comment.
reduce로 순환하면서 타입을 체크하고 진행하는 방법도 있습니다!
There was a problem hiding this comment.
오..... reduce도 괜찮겠네요!
간단한 건 로그인 여부를 체크한 다음, 현재 디바이스를 동적으로 트래킹하는 방법이 가장 쉽습니다. 문제는 다양한 디바이스에 보내는 건 힘들 거 같네요. 한 디바이스에라도 정확하게 보내게 구현할까요?
시간이 오래걸린다... 이건 클라이언트 사이드의 service worker를 수정해서 api를 보내도록 만들어야 겠네요! |
close #341
✅ 작업 내용
📌 이슈 사항
✍ 궁금한 점
😎 체크 사항