Skip to content

불씨 #20

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
Open
Leebeom-seok wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

불씨 #20

Leebeom-seok wants to merge 4 commits into from

Conversation

@Leebeom-seok
Copy link
Contributor

@Leebeom-seok Leebeom-seok commented Nov 17, 2023

📌 관련 이슈

세션, 채팅

✨ 과제 내용

  1. 프론트에서 로그인 가능
  2. individual에서 왼쪽 room목록 불러옴
  3. room목록 선택후 채팅하면 잘 돌아감

문제점

  1. room목록을 불러움 -> 친구목록은 아님(하지만 친구 목록 불러오는건 구현하긴함 / 콘솔로그로 찍힘)
  2. individual room목록 컨포넌트 렌더링에 문제점이 있음 ( vscode에 주석 같은거 하나 적고 저장 같은 방법으로 사용해야함;)

📸 스크린샷

image

📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들

음 한지 오래되서 잘 기억은 안나는데
백에서 db쿼리 짤때 여러 테이블에서 특정 값들을 뽑아서(select) 사용할 때 DTO를 Class가 아닌 Interface로 만들어야 한다고 해서 그렇게 했습니당

@Leebeom-seok
프론트에서 세션사용가능
a90938c 
왼쪽 채팅 리스트 가능하게..(근데 시험용으로 한 임시 user의 id나 room id 이런건 수정해야함)
@Leebeom-seok
프론트에서 세션사용가능
a790b35 
왼쪽 채팅 리스트 가능하게..(근데 시험용으로 한 임시 user의 id나 room id 이런건 수정해야함)
@Leebeom-seok
프론트에서 세션사용가능
d231c4a 
왼쪽 채팅 리스트 가능하게..(근데 시험용으로 한 임시 user의 id나 room id 이런건 수정해야함)
@Leebeom-seok
프론트에서 세션사용가능
ad28bc0 
왼쪽 채팅 리스트 가능하게..(근데 시험용으로 한 임시 user의 id나 room id 이런건 수정해야함)
왼쪽 채팅 리스트가 친구목록은 아니고 현재 가지고 있는 채팅목록임
문제점은 컨포넌트 렌더링 문제가 있어서 individual 들어간다음 주석같은거로 .하나 입력하고 저장해서 랜더링 다시 해야 작동함 ㅋㅁㅋ
@mukhoplus mukhoplus requested review from mukhoplus, richsubin and skybluelion and removed request for mukhoplus and skybluelion November 18, 2023 04:26
@mukhoplus mukhoplus added the Feat 기능 개발 label Nov 18, 2023
mukhoplus
mukhoplus approved these changes Nov 18, 2023
View reviewed changes
Copy link
Contributor

@mukhoplus mukhoplus left a comment •
edited
Loading

Choose a reason for hiding this comment

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

  1. 기존 코드랑 내용이 다른거 같은데 확인 부탁 드립니다.
  2. 문제점은 이슈에 추가해주새오.
  3. 추후에는 다음과 같은 사항을 유의해주새오.
  • 코드 서식(띄어쓰기 등)
  • 출력문 제거(추후 기능 추가 부분이 아닌 경우)
  • 커밋 메시지 서식
  • PR 제목

skybluelion
skybluelion approved these changes Nov 18, 2023
View reviewed changes
Copy link
Member

@skybluelion skybluelion left a comment

Choose a reason for hiding this comment

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

오랜만이네요. 고생 많았습니다.
SESSION 부분이 수정됐던데, 이제 로그인 세션 유지 되는건가요?

@mukhoplus
Copy link
Contributor

mukhoplus commented Nov 18, 2023

질문

  1. 세션의 유지 시간은 얼마인가요?
  2. 세션이 만료되면 소켓 세션도 같이 만료되나요? 세션이 만료되었을 때 채팅 전송을 하면 어떤 동작이 발생하나요?

@mukhoplus mukhoplus self-requested a review November 18, 2023 05:37
mukhoplus
mukhoplus requested changes Nov 18, 2023
View reviewed changes
Copy link
Contributor

@mukhoplus mukhoplus left a comment

Choose a reason for hiding this comment

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

코드 다른 부분 있으니 확인과 테스트 후 코멘트 달아주세요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mukhoplus mukhoplus mukhoplus requested changes

@skybluelion skybluelion skybluelion approved these changes

@richsubin richsubin Awaiting requested review from richsubin

Requested changes must be addressed to merge this pull request.

Assignees

@Leebeom-seok Leebeom-seok

Labels

Feat 기능 개발

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Leebeom-seok @mukhoplus @skybluelion