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

Refector: Store의 Chat, Socket 관련한 부분 리팩토링이 필요합니다. #33

Open
soulchicken opened this issue Jul 31, 2023 · 0 comments
Labels
feature New feature or request refactoring refactoring

Comments

@soulchicken
Copy link
Member

우리 성하가 너무 멋진 기능을 완성시키면서 소켓 통신을 뚫었습니다.

리팩토링 이유
socket.ts 파일의 책임이 막중하기에 리팩토링이 필요합니다.
채팅이 여러번 들어왔을 때 기존 API 방식의 경우 요청을 보내고 응답을 기다리는 동안 loading state가 변하지 않아서 모달이 잘 동작했지만
소켓 방식의 경우 요청과 응답을 받는 과정이 분리되어있기에 loading state를 변하게 하는 방식은 변화가 필요합니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request refactoring refactoring
Projects
None yet
Development

No branches or pull requests

1 participant