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

[Epic#15, Epic#41] 채팅 기능 보완 #55

Closed
4 tasks done
devphilip21 opened this issue Nov 27, 2019 · 0 comments
Closed
4 tasks done

[Epic#15, Epic#41] 채팅 기능 보완 #55

devphilip21 opened this issue Nov 27, 2019 · 0 comments
Assignees
@devphilip21
Labels
🐣Feature 새로운 기능을 추가할 때 사용하는 라벨
Projects
Sprint - 03 🏃‍♂️
Milestone
Sprint - 03

Comments

@devphilip21
Copy link
Contributor

devphilip21 commented Nov 27, 2019 •
edited
Loading

[Epic#15, Epic#41] 채팅 기능 보완

  • 2주차 스프린트 때 완료한 채팅기능이 미흡한 부분이 있어서 보완해야한다.
  • 채팅이 쭉 쌓일 때, 스크롤바의 상태가 가장 최근 채팅을 볼 수 있도록 맞춰져야 한다.
  • 처음 화면 진입시 기존의 채팅들을 불러와야 한다.
  • 채팅을 하면 인풋창이 초기화 되어야 한다.
  • 채팅창에서 엔터키를 이용해도 전송이 가능하다.

완료 조건 ✅

  • 채팅 전송시 Input 창 클리어
  • 채팅 Input에서 엔터키를 이용해서 전송이 가느해야한다.
  • 화면 진입시 기존의 채팅로그를 모두 불러온다.
  • 채팅이 올라오면 가장 최근의 채팅이 보여야한다.

관련 이슈 📎

레퍼런스 📚

레퍼런스 없음
@devphilip21 devphilip21 added the 🐣Feature 새로운 기능을 추가할 때 사용하는 라벨 label Nov 27, 2019
@devphilip21 devphilip21 added this to the Sprint - 03 milestone Nov 27, 2019
@devphilip21 devphilip21 self-assigned this Nov 27, 2019
devphilip21 pushed a commit that referenced this issue Nov 27, 2019
[ FEATURE ] 채팅 전송시 Input 클리어 및 엔터키로 전송 (#55)
6816cf8 
Why
- UX 측면에서 엔터키를 이용해 전송이 가능해야한다.
- 전송이 되었다면 채팅창이 클리어되어야한다.

What
- ChatInput 컴포넌트에서 채팅 전송시 message state 초기화
- ChatInput 컴포넌트에서 Enter Key 에 대한 이벤트 추가
- ChatLogs 에서 엔터키 적용 가능하도록 수정
devphilip21 pushed a commit that referenced this issue Nov 27, 2019
[ FEATURE ] 채팅 로그를 가져오는 API 완성 (#55)
4c93931 
Why
- 페이지 진입시 기존의 채팅을 모두 가져와야한다.

What
- getChatLogs Query 추가
- model 을 payload 로 변경하는 부분에 대해 메서드 생성
devphilip21 pushed a commit that referenced this issue Nov 27, 2019
[ FEATURE ] 채팅 로그를 가져오는 Apollo Cache 기능 완성 (#55)
afc47f5 
Why
- 페이지 진입시 기존의 채팅을 모두 가져와야한다.

What
- 페이지 진입시 채팅로그를 모두 가져와 캐시에 저장하고, broadcast 한다.
- broadcast 된 쿼리를 받기 위해서, readQuery 대신 useQuery 를 활용한다.
devphilip21 pushed a commit that referenced this issue Nov 27, 2019
[ FEATURE ] 채팅시 채팅로그 UI가 가장 최근의 채팅을 따라가도록 한다. (#55)
622b23e 
Why
- 채팅은 커뮤니케이션을 위한 것으로 실시간 서로의 의견을 볼 수 있어야한다.
- 따라서 최근의 채팅이 계속 보이는 것이 적절한 UI/UX 이다

What
- 스크롤 마지막 부분을 계산하는 dom Util 추가
- 채팅 로그가 올라올 때, 스크롤 마지막으로 이동
@devphilip21 devphilip21 mentioned this issue Nov 27, 2019
Merged
devphilip21 pushed a commit that referenced this issue Nov 27, 2019
[ REFACTOR ] 리뷰사항 반영 (#55)
23e6039 
Why
- 스타일드 컴포넌트 네이밍 방식을 통일할 필요가 있다.
- 불필요한 코드가 삽입되어 있다.

What
- if (event) 삭제
- 컴포넌트 이름과 최상위 스타일드 컴포넌트 이름을 통일시킨다.
aereeeee added a commit that referenced this issue Nov 27, 2019
@aereeeee
[ MERGE ] 채팅 기능 보완 (#55)
80f1d6d 
* feature/issue55 -> develop
@devphilip21 devphilip21 mentioned this issue Nov 28, 2019
Closed
3 tasks
@always-awake always-awake mentioned this issue Nov 28, 2019
Merged
@devphilip21 devphilip21 mentioned this issue Nov 29, 2019
Closed
@aereeeee aereeeee mentioned this issue Dec 2, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@devphilip21 devphilip21

Labels
🐣Feature 새로운 기능을 추가할 때 사용하는 라벨
Projects
No open projects
Sprint - 03 🏃‍♂️
Done
Milestone
Sprint - 03
Development

No branches or pull requests
1 participant
@devphilip21