-
Notifications
You must be signed in to change notification settings - Fork 2
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
게시글 실시간으로 보여지기 #140
Comments
WooYeonSeo
added a commit
that referenced
this issue
Dec 2, 2019
why 새로운 글을 등록하면 구독자들에게 알려주고, 구독자들은 그 알람을 받아야 함 how graphql의 pubsub을 활용해서 새로 등록된 글에 대한 알람이 오늘 것을 확인 주의 서버쪽 구현이며 아직 필터링은 되지 않음
WooYeonSeo
added a commit
that referenced
this issue
Dec 4, 2019
why 디비에서 비교할 때 datetime string으로 비교하기 위해서 how 같은 형태의 string으로 변환하는 util함수 작성
WooYeonSeo
added a commit
that referenced
this issue
Dec 4, 2019
why 다른 사람이 등록한 글을 새로고침 해야 알 수 있었어서 실시간으로 새로운 글을 볼 수 있도록 등록 how apollo pubsub을 이용해서 새로 입력한 글이 보여짐 apollo subscribtomore을 이용해서 data를 merge 주의 아직 친구만 필터링 되는 부분은 없음
WooYeonSeo
added a commit
that referenced
this issue
Dec 4, 2019
친구가 등록한 글만 피드에 뜨게 해야 하기 때문에 친구에 대해서만 새 피드가 뜨도록 설정
WooYeonSeo
added a commit
that referenced
this issue
Dec 5, 2019
친구 찾기가 양방향 검색이 아니어서 수락한 사람의 친구에 검색되지 않음 양방향으로 체크하게 변경
WooYeonSeo
added a commit
that referenced
this issue
Dec 5, 2019
why 피드를 스크롤링 하다가 새로운 피드가 쌓이면 사용자가 알 수 있게 하기 위해서 피드 등록 알림 기능 개발 how 새로운 피드가 피드 피드 기준 상수보다 많고, 윈도우가 제일 상단이 아닐때 새로운 피드 등록 알림 새로운 등록 알림은 상단으로 이동해서 보기 전까지 사라지지 않음. pubsub으로 사용자의 친구들만 피드알림을 받는다.
WooYeonSeo
added a commit
that referenced
this issue
Dec 5, 2019
apoll requet 쿼리에 썸네일 누락으로 기본이미지만 떠서 수정
WooYeonSeo
added a commit
that referenced
this issue
Dec 5, 2019
댓글이 없을 때에도 새 댓글이 입력 가능해야 함 그래서 컴포넌트 분리
WooYeonSeo
added a commit
that referenced
this issue
Dec 5, 2019
새글이 피드에 등록되어야 하고 새 피드가 알람이 와야함. 새 글 알림을 payload로 전달받은 등록자와 subscribe 에 등록된 사용자의 email을 비교해서 알림 발송
WooYeonSeo
added a commit
that referenced
this issue
Dec 5, 2019
feed인데 comment subscription을 등록하는 함수처럼 이름을 지어서 변경함
WooYeonSeo
added a commit
that referenced
this issue
Dec 5, 2019
WooYeonSeo
added a commit
that referenced
this issue
Dec 5, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The text was updated successfully, but these errors were encountered: