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

[Fix] 서버와 채팅 메시지 타입 맞지 않아 발생했던 error 해결 & 리팩토링 #109

Merged
merged 3 commits into from
Aug 21, 2024

Conversation

nim-od
Copy link
Member

@nim-od nim-od commented Aug 21, 2024

🔘Part

  • 이벤트 페이지


🔎 작업 내용

as-is

기획 단계에서의 채팅기능에 따라 - notice, block, chat 이 배열에 섞여서 내려옴
�준하님과 채팅 관련 기능 구현 초반에 논의했던 데이터 타입

export type NoticeChatProps = {
	id: string;
	type: 'n';
	content: string;
	sender?: never;
	team?: never;
};

export type MessageChatProps = {
	id: string;
	type: 'm';
	sender: string;
	content: string;
	team: SocketCategory;
};

type BlockedChatProps = {
	id: string;
	type: 'b';
	sender?: never;
	content?: never;
	team?: never;
};

export type ChatProps = NoticeChatProps | MessageChatProps | BlockedChatProps;

to-be

  1. 공지는 상단 고정되어 보여주기에 따로 관리
  2. 소켓 연동 시점에 채팅 리스트를 받아온다.
    - 서버는 이미 block된 채팅을 제외하고, 메시지 내역과 notice를 �리스트 형태로 보내준다.
    - notice는 채팅 내역의 마지막에 단 하나만 포함된다.
    - notice가 없을 경우 내려오지 않으며, 마지막 데이터의 공지 여부는 sender로 판단한다.

@nim-od nim-od self-assigned this Aug 21, 2024
@nim-od nim-od merged commit 8b9bdf7 into main Aug 21, 2024
@nim-od nim-od deleted the fix-chat-error branch August 21, 2024 13:57
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.

1 participant