Skip to content

[#209] Refactor: 채팅 목록, 채팅방 페이지 개선#212

Open
Heeeera wants to merge 5 commits into
mainfrom
#209/refactor/chatting/CHOI
Open

[#209] Refactor: 채팅 목록, 채팅방 페이지 개선#212
Heeeera wants to merge 5 commits into
mainfrom
#209/refactor/chatting/CHOI

Conversation

@Heeeera

@Heeeera Heeeera commented May 17, 2024

Copy link
Copy Markdown
Contributor

💬 Issue Number

closes #209

🤷‍♂️ Description

채팅 목록, 채팅방 페이지에서 불필요한 API 호출을 제거했습니다.

  • 새로운 메시지가 올 때마다 채팅 내역 불러오던 API 호출 제거
  • 새로운 메시지가 올 때마다 sender를 구분하기 위해 로그인한 사용자 id를 받아오는 API 호출 제거
  • 채팅 목록 페이지에서 최신 메시지, 읽지 않은 메시지를 채팅 목록 API 응답으로 내려 받으면서 추가적인 API 호출 제거

📋 Check List

PR 전 체크해주세요.

  • Merge 하는 브랜치가 올바른가?
  • 코딩컨벤션을 준수하는가?
  • PR과 관련없는 변경사항이 없는가?

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying boardsignal with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8a8ca3a
Status: ✅  Deploy successful!
Preview URL: https://4a34c5ae.team-civilwar-boardsignal-fe.pages.dev
Branch Preview URL: https://209-refactor-chatting-choi.team-civilwar-boardsignal-fe.pages.dev

View logs

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.

[Refactor] 채팅 목록, 채팅방 기능 개선

1 participant