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

[add] 채팅방 입장시 dto에 유저 정보 추가 #163

Merged
merged 5 commits into from
Feb 3, 2025
Merged

[add] 채팅방 입장시 dto에 유저 정보 추가 #163

merged 5 commits into from
Feb 3, 2025

Conversation

dyk-im
Copy link
Contributor

@dyk-im dyk-im commented Feb 3, 2025 •
edited
Loading

개요

  1. 모임 상세 조회시 채팅방 id도 조회되도록 수정(모임과 채팅방의 id가 차이나는 경우 발생)
  2. 채팅방 입장 시 유저 id, 역할도 조회되록 수정

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

📣 To Reviewers

@dyk-im dyk-im requested a review from a team February 3, 2025 10:54
@dyk-im dyk-im linked an issue Feb 3, 2025 that may be closed by this pull request
[add] 채팅방 입장시 DTO에 유저 정보 추가 #162
Closed
2 tasks
@dyk-im dyk-im self-assigned this Feb 3, 2025
@dyk-im dyk-im added add feat 이외의 부수적인 코드, 파일, 라이브러리 추가 동윤 labels Feb 3, 2025
ehs208
ehs208 approved these changes Feb 3, 2025
View reviewed changes
Copy link
Member

@ehs208 ehs208 left a comment

Choose a reason for hiding this comment

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

크게 수정사항도 없어보이고 급한 사안인거 같아 Approve 드리겠습니다!

jj0526
jj0526 approved these changes Feb 3, 2025
View reviewed changes
Copy link
Contributor

@jj0526 jj0526 left a comment

Choose a reason for hiding this comment

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

문제 없어보입니다! 수고하셨어요

seokjun01
seokjun01 approved these changes Feb 3, 2025
View reviewed changes
Copy link
Contributor

@seokjun01 seokjun01 left a comment

Choose a reason for hiding this comment

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

코드 확인하였습니다! 바로 승인하겠습니다.

@dyk-im dyk-im merged commit 74a0efd into dev Feb 3, 2025
2 checks passed
@ehs208 ehs208 deleted the add/162-채팅방-입장시-dto에-유저-정보-추가 branch February 4, 2025 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jj0526 jj0526 jj0526 approved these changes

@ehs208 ehs208 ehs208 approved these changes

@seokjun01 seokjun01 seokjun01 approved these changes

Assignees

@dyk-im dyk-im

Labels
add feat 이외의 부수적인 코드, 파일, 라이브러리 추가 동윤
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[add] 채팅방 입장시 DTO에 유저 정보 추가
4 participants
@dyk-im @jj0526 @ehs208 @seokjun01