[Run/open-live-session] - 실시간 대기방 생성 API 개발#77
Merged
Conversation
Collaborator
|
ㅓㅇ으ㅡ악 지금 구현중이라 코드 리뷰가 늦어져 죄송합니다. |
Collaborator
|
리뷰에 앞서,,, 컨벤션이 자주 바뀌어서 사실 맞추고 따라가는데 많은 어려움을 겪고있는데요. |
Collaborator
Author
@hughesgoon |
4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
연관된 이슈를 적어주세요 📌
작업한 내용을 설명해주세요 ✔️
생성한 세션에 대해 실시간 달리기 방을 만드는 API입니다.
요구사항은 다음과 같습니다.
구현 사항
트러블 슈팅
@CurrentUser사용에서 유저 아이디를 받지 못하고 익명유저로 id가 넘어가고 있습니다.ThreadLocal 한 객체인 Authentication을 jwt filter에서 세팅하는 로직이 없어서 그런 것 같습니다.
리뷰어에게 하고 싶은 말을 적어주세요
확인하기
System.out을 제거했나요?