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

Feat/retrieve chat list #59

Merged
merged 55 commits into from
Sep 24, 2023
Merged

Feat/retrieve chat list #59

merged 55 commits into from
Sep 24, 2023

Conversation

seongyunlee
Copy link
Contributor

업데이트 유형

  • Hot Fix
  • Release
  • Develop
  • Others

업데이트 개요

  • Fix # (issue)
  • Feature # (issue)
    [여기에 PR 한줄 요약 작성, 대상 이슈번호 작성.]

업데이트 요약

업데이트 내역 작성

해결한 문제

해결한 문제 등 이슈

미해결 문제

미해결 이슈, 발견한 이슈

테스트

  • 유닛 테스트
  • 빌드 테스트 (통합 테스트)
  • 기타 유효성 테스트

수정 사항 진단

  • 코드가 이 프로젝트의 스타일 지침을 따릅니다.
  • 코드에 대한 자체 리뷰를 수행했습니다.
  • 변경 내역에 대해 주석을 작성했습니다.
  • 해당 PR에 대한 문서를 변경했습니다.
  • 기능, 수정 사항, 이슈에 대한 테스트 코드를 추가했습니다.
  • 변경 내용이 로컬 개발환경에서의 테스트를 통과했습니다.

채팅 가져오기 api 리팩토링.
-학생이 일반 질문 올렸을 때 그 질문도 채팅리스트에 보이게하기

# Conflicts:
#	src/chatting/chatting.repository.ts
#	src/chatting/chatting.service.ts
#	src/chatting/entities/chatting.interface.ts
#	src/user/user.repository.ts
…estion

# Conflicts:
#	src/chatting/chatting.service.ts
#	src/offer/offer.repository.ts
#	src/offer/offer.service.ts
#	src/question/entities/question.interface.ts
#	src/question/entities/question.schema.ts
#	src/question/question.repository.ts
# Conflicts:
#	src/offer/offer.service.ts
#	src/tutoring/tutoring.controller.ts
#	src/tutoring/tutoring.service.ts
# Conflicts:
#	src/chatting/chatting.service.ts
Copy link
Member

@w8385 w8385 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG GOOD

@seongyunlee seongyunlee merged commit d12501e into dev Sep 24, 2023
@seongyunlee seongyunlee deleted the feat/retrieve_chat_list branch September 24, 2023 08:16
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.

2 participants