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.

Sign up for GitHub

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

Jump to bottom

[feature/socket] 소켓 통신을 이용한 채팅 코드 재구성 #60

Open
3 tasks done
hobiJeong opened this issue Oct 20, 2023 · 0 comments
Open
3 tasks done

[feature/socket] 소켓 통신을 이용한 채팅 코드 재구성 #60

hobiJeong opened this issue Oct 20, 2023 · 0 comments
Assignees
@hobiJeong

Comments

@hobiJeong
Copy link
Collaborator

hobiJeong commented Oct 20, 2023 •
edited
Loading

  • 컨트롤러 - 서비스 - 레포지터리 - socket 순이 아닌 socket - 서비스 - 레포지터리 순으로 변경
  • socket 에러 핸들링 추가
  • image도 변경
@hobiJeong hobiJeong self-assigned this Oct 20, 2023
hobiJeong added a commit that referenced this issue Oct 20, 2023
@hobiJeong
feat(#60): socket-service-repository 순으로 채팅 로직 변경
97393d7
hobiJeong added a commit that referenced this issue Oct 20, 2023
@hobiJeong
feat(#60): socket.emit은 gateway에서 실행하도록 변경
d2211a9
hobiJeong added a commit that referenced this issue Oct 20, 2023
@hobiJeong
feat(#60): 테스트용 코드 삭제
fc2ff6a
hobiJeong added a commit that referenced this issue Oct 20, 2023
@hobiJeong
feat(#60): controller 채팅생성 삭제
95eb9de
hobiJeong added a commit that referenced this issue Oct 20, 2023
@hobiJeong
feat(#60): Dto에 imageUrl추가, gateway에서 imageUpload 추가(이미지 업로드는 아직 미완)
b3dbf18
hobiJeong added a commit that referenced this issue Oct 21, 2023
@hobiJeong
feat(#60): swagger-custom-decorators 추가, 사용하지 않는 코드들 삭제(Socket.io 및 챗…
1b91b28 
… 이미지 생성은 아직 미구현으로 추가하지 않음)
hobiJeong added a commit that referenced this issue Oct 21, 2023
@hobiJeong
refactor(#60): 추가 삭제
5a9469f
hobiJeong added a commit that referenced this issue Oct 21, 2023
@hobiJeong
modify(#60): synchronize: true로 변경
b90d461
hobiJeong added a commit that referenced this issue Oct 22, 2023
@hobiJeong
refactor(#60): 피드백 내용 반영 및 수정
3e53c29
hobiJeong added a commit that referenced this issue Oct 23, 2023
@hobiJeong
fix(#60): IsNotEmpty, IsOptional 둘 다 들어간 부분 수정
378f80c
hobiJeong added a commit that referenced this issue Oct 23, 2023
@hobiJeong
fix(#60): Dto에서 ?를 붙일 시 Service에 코드를 넘겨줄 때 오류가 생겨서 ? 제거
daf24ec
hobiJeong added a commit that referenced this issue Oct 23, 2023
@hobiJeong
modify(#60): dto 수정
fa25124
hobiJeong added a commit that referenced this issue Oct 23, 2023
@hobiJeong
feat(#60): 읽지 않은 메세지 개수 가져오기 추가
70e0b9e
hobiJeong added a commit that referenced this issue Oct 25, 2023
@hobiJeong
feat(#60): socket validationpipe, exception-filter 적용. event기반 메서드 ap…
6bb1daf 
…i 문서 작성 위해 asyncapi 추가, loignDto추가
hobiJeong added a commit that referenced this issue Oct 25, 2023
@hobiJeong
feat(#60): 이유는 모르겠으나 asyncapi를 따로 함수로 만들어서 쓰면 문서 작성이 되지 않아서 삭제 및 main…
c4cb49c 
….ts에서 생성
hobiJeong added a commit that referenced this issue Oct 31, 2023
@hobiJeong
refactor(#60): 챗이미지 생성 테스트 코드
702b218
hobiJeong added a commit that referenced this issue Oct 31, 2023
@hobiJeong
refactor(#60): await 제거
5d60240
hobiJeong added a commit that referenced this issue Oct 31, 2023
@hobiJeong
refactor(#60): loign에서 Roomid Validation 문제로 위해 IsArray로 변경, 채팅 이미지 업…
d01431e 
…로드 로직 재구성
hobiJeong added a commit that referenced this issue Oct 31, 2023
@hobiJeong
feat(#60): if문으로 함수 실행되도록 수정
8ae5060
hobiJeong added a commit that referenced this issue Oct 31, 2023
@hobiJeong
refactor(#60): 임시 마무리
b090bd5
hobiJeong added a commit that referenced this issue Nov 1, 2023
@hobiJeong
fix(#60): 이미지 전송 로직 오류 수정
06be1e9
hobiJeong added a commit that referenced this issue Nov 7, 2023
@hobiJeong
feat(#60): Guard, 유저 id 받아오는 커스텀 데코레이터 추가, chat-notification.schema 수정
98a5c38
hobiJeong added a commit that referenced this issue Nov 7, 2023
@hobiJeong
feat(#60): chat.schema수정(isSeen칼럼 추가), chat-notification.schema 삭제(ch…
42fe5bc 
…at.schema로 병합) 및 채팅 알람 가져올 때 읽지않은 가장 최근의 메세지 1개와 읽지않은 메세지의 총 개수 불러오는 로직으로 수정
hobiJeong added a commit that referenced this issue Nov 7, 2023
@hobiJeong
refactor(#60): 기존의 읽지않은 메세지 갯수만 가져오는 로직 임시 주석처리(아마 삭제될 예정)
5c9479b
hobiJeong added a commit that referenced this issue Nov 7, 2023
@hobiJeong
refactor(#60): 쓸모없는 주석 삭제
2affa2f
hobiJeong added a commit that referenced this issue Nov 7, 2023
@hobiJeong
refactor(#60): service, repository 코드 분리
6fc1c75
hobiJeong added a commit that referenced this issue Nov 7, 2023
@hobiJeong
refactor(#60): return값 type unknown이어서 dto로 타입 지정
3e3e827
hobiJeong added a commit that referenced this issue Nov 7, 2023
@hobiJeong
modify(#60): createdAt필드 따로 정의하지 않아도 되도록 수정(기능상 문제를 고침), updatedAt삭제 …
9e41f2d 
…및 dto도 맞춰서 수정
hobiJeong added a commit that referenced this issue Nov 7, 2023
@hobiJeong
feat(#60): chat.schema에서 다시 updatedAt 필드 추가, 채팅 조회 시 userId도 같이 받도록 수…
8df2127 
…정, Service코드에서 해당 유저 채팅창 속해 있는지 검사 로직 추가, 채팅 조회 시 isSeen필드 값 true 값으로 바꾸는 로직 추가, 채팅 내역 없을 시 에러를 뱉는게 아니라 빈 배열 return 하도록 수정
hobiJeong added a commit that referenced this issue Nov 7, 2023
@hobiJeong
refactor(#60): 채팅 조회 시 isSeen값 업데이트 할 때 false인 값들만 update 하도록 수정
1446835
hobiJeong added a commit that referenced this issue Nov 7, 2023
@hobiJeong
modify(#60): 채팅방 전체 조회 시 값이 없더라도 에러를 뱉지 않고 빈배열을 반환하도록 수정 및 decorator추…
9d7303c 
…가, 수정
hobiJeong added a commit that referenced this issue Nov 7, 2023
@hobiJeong
modify(#60): 예외처리 변경에 맞춰 swagger문서 수정, 알람 및 image생성 swagger 문서 생성
4cedf61
hobiJeong added a commit that referenced this issue Nov 8, 2023
@hobiJeong
modify(#60): 소켓 테스트용 client.js 삭제
5139e75
hobiJeong added a commit that referenced this issue Nov 16, 2023
@hobiJeong
modify(#60): guard 수정
9ff9819
hobiJeong added a commit that referenced this issue Nov 16, 2023
@hobiJeong
modify(#60): rxjs 문법 에러 수정
1fb7982
hobiJeong added a commit that referenced this issue Nov 21, 2023
@hobiJeong
feat(#60): add interceptors for set success response type in chat-con…
f19b80c 
…troller and socket-gateway
hobiJeong added a commit that referenced this issue Nov 21, 2023
@hobiJeong
feat(#60): add ws-guard(not working)
28ba6dd
hobiJeong added a commit that referenced this issue Nov 21, 2023
@hobiJeong
refactor(#60): change return variable's name
0b32ab3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hobiJeong hobiJeong

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@hobiJeong