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

feat(chat): 채팅 직접 보낼 수 있도록 Handler 등록 및 scroll 구현 #68

Merged
merged 2 commits into from
Jul 23, 2024

Conversation

seungholee-dev
Copy link
Contributor

@seungholee-dev seungholee-dev commented Jun 23, 2024

개요

  • 어플 내부에서 채팅을 직접 보낼 수 있도록 Handler를 등록한다.

수정 사항

  • handleSend 핸들러 함수를 onPress에 채팅을 보냄
  • useRef를 사용해서 현재 최신 채팅을 기준으로 스크롤될 수 있도록 구성
    (FlatList의 최신으로 갈 수 있게끔)

@seungholee-dev
Copy link
Contributor Author

seungholee-dev commented Jun 26, 2024

임시 PR 입니다 :) 작성 완료 후 Review Request 할게요!

@seungholee-dev seungholee-dev force-pushed the feat/chat-display-name branch 2 times, most recently from c30e8ff to 5b85ca7 Compare July 1, 2024 10:36
Base automatically changed from feat/chat-display-name to prod July 1, 2024 10:36
@seungholee-dev seungholee-dev force-pushed the feat/send-chat branch 2 times, most recently from a44f02b to c82e409 Compare July 1, 2024 10:46
@seungholee-dev
Copy link
Contributor Author

seungholee-dev commented Jul 1, 2024

테스트 방법

  1. 웹소켓을 열어서 (apic.com), 채팅을 버튼을 눌러 전송해봅니다.
    image

  2. 채팅방에서 버튼을 눌러서 본인 채팅이 전송이되는 것을 확인합니다.

@seungholee-dev
Copy link
Contributor Author

seungholee-dev commented Jul 5, 2024

Rebase 완료하여 코드 확인부탁드려요!
@nnyouung

@seungholee-dev seungholee-dev changed the base branch from prod to ci/add-pr-checks July 5, 2024 13:24
@seungholee-dev seungholee-dev force-pushed the feat/send-chat branch 2 times, most recently from 731f3f7 to 8f1da52 Compare July 5, 2024 13:27
Base automatically changed from ci/add-pr-checks to prod July 10, 2024 13:20
@seungholee-dev
Copy link
Contributor Author

seungholee-dev commented Jul 11, 2024

뒤에 연이은 PR이 많아서, 이 PR의 리뷰는 잠시 스킵해주시고 이후 PR들을 먼저 리뷰 해주시면
이후에 리베이스 후, 리뷰 요청 다시 드릴게요! @nnyouung

@seungholee-dev seungholee-dev removed the request for review from nnyouung July 11, 2024 12:33
@seungholee-dev seungholee-dev force-pushed the feat/send-chat branch 5 times, most recently from ae6ec92 to 18d7a51 Compare July 18, 2024 15:03
@seungholee-dev seungholee-dev requested a review from nnyouung July 18, 2024 15:05
개요

- 어플 내부에서 채팅을 직접 보낼 수 있도록 Handler를 등록한다.

수정 사항

- handleSend 핸들러 함수를 onPress에 채팅을 보냄
- useRef를 사용해서 현재 최신 채팅을 기준으로 스크롤될 수 있도록 구성
(FlatList의 최신으로 갈 수 있게끔)
개요

- iOS에서 웹소켓 사용 시 필요로 하는 모듈인 text-encoding이 누락되어
이를 추가하여 고친다. (이 부분을 제외하면 iOS에서 stompjs 사용 불가)
- React Native 프로젝트에서는 아래 링크에서 말하는대로 이 부분이
필수적으로 필요함

- 추가로, 이전 병합 과정에서 빠진 stompjs와 expo secure storage npm
package를 재추가한다

아래 링크 참고:
https://stomp-js.github.io/workaround/stompjs/rx-stomp/react-native-additional-notes.html
@seungholee-dev seungholee-dev merged commit 399f79d into prod Jul 23, 2024
1 check passed
@seungholee-dev seungholee-dev deleted the feat/send-chat branch July 23, 2024 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants