codestates / codestates/Reciper-client
[✍️ Dev Log] 이우성 / 2021-06-15
- Dominant language
- TypeScript
- Stars
- 13
- Forks
- 7
- PR merge metrics
- No merged PRs in 30d
Description
### 오늘은 어떻게 프로젝트에 기여했나요?
- #416 채팅 삭제 및 수정 기능 성능개선을 위해 전면 Redux로 리팩토링
채팅 삭제 및 수정 기능을 구현하기 위해 Chat 데이터를 Props를 통해 컴포넌트로 전달해 주고 있었다. (deps: 2)
컴포넌트를 두 번이나 관통해 내려가서 채팅 데이터의 Props를 겨우 전달 받는데, 이러면 2번을 거치면서 강제 렌더링이 되기 때문에 성능에 매우 안좋아 보인다.
처음 프로젝트를 할 때는 소켓 구현을 위해 일단 state로 관리를 했지만, 구현에 성공했기 때문에 현재 Redux로 리팩토링을 해서 상태관리를 다시 구현한 상태이다.
### 오늘의 프로젝트에서 힘든 점은 무엇인가요?
- 현재 수정 및 삭제기능을 Redux로 리팩토링에 성공했지만 채팅 Room을 옮길 때 이슈가 발생했다.
방을 옮겨 다닐때 현재 방의 데이터를 다 이끌고 다른 방으로 가지는 이슈가 발생했다. state로 관리를 했을 때 처럼 이슈를 해결해 보려고 했지만 현재 이슈를 해결하지 못한 상태이다.....
이슈를 어서 끝마치고 merge 하고 싶다....
### 내일은 프로젝트에 기여하기 위해 무엇을 해야 하나요?
- [x] 채팅 전체적으로 코드 점검
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.