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.

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

[Merge] #123 topic & timer push notification #126

Merged
merged 1 commit into from Jun 19, 2022

Conversation

ccdkk
Copy link
Collaborator

@ccdkk ccdkk commented Jun 19, 2022

작업 내용

  • 깃이 꼬여버리는 바람에 develop 에서 다시 브랜치파서 작성했던 코드 복붙했습니다.
  • topic에 푸시노티피케이션 추가
  • timer에 푸시노티피케이션 추가
  • firebase에 저장된 date값을 이용하여, topic information을 저장하고 fetch하는 함수에 넣으려고 했으나, stored, fetch함수가
    너무 빈번하게 일어나서 포기하고 조건이 충족되는 순간의 Date()로 바로 보내게 구현하였습니다.

리뷰 포인트

  • 다른 유저가 topic을 올렸을 때 다른 사람의 폰에서도 알림이 잘 오는지,
  • topic을 올린 사람의 폰에서도 알림이 잘 오는지,
  • timer뷰에서 3분전, 0분에 알림이 잘 가는지
  • 현재 테스트를 위해서 picker의 minute범위를 0분부터 시작하게 해놨습니다. 테스트 다 해보시고 5분으로 바꿔야할 수 있습니다!
  • 다른 휴대폰에서 토픽이 연결되려면 서로 한 번씩은 던지고 나서 연결되는 것 같습니다. notification도 연결된 조건을 가져오기 때문에
    서로 연결이 되고 나서야 두 휴대폰에 모두 정상적으로 알림이 가는 것 같습니다

다음으로 진행될 작업

질문

References

- topic에 푸시노티피케이션 추가
- timer에 푸시노티피케이션 추가
@ccdkk ccdkk added this to In progress in MC2 via automation Jun 19, 2022
Copy link
Collaborator

@imparang imparang left a comment

Choose a reason for hiding this comment

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

확인이요

@imparang imparang merged commit cbde27d into develop Jun 19, 2022
MC2 automation moved this from In progress to Done Jun 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
MC2
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants