Skip to content

Conversation

@ekgns33
Copy link
Collaborator

@ekgns33 ekgns33 commented Jan 11, 2025

연관된 이슈를 적어주세요 📌

작업한 내용을 설명해주세요 ✔️

  • 생성한 세션에 대해 실시간 달리기 방을 만드는 API입니다.

  • 요구사항은 다음과 같습니다.

    1. 시작시간 10분 전에는 세션글 작성자만 대기방을 열 수 있음.
    2. 시작시간 이후 5분까지는 세션글 작성자가 대기방을 열 수 있음.
    3. 시작시간 5분 이후부터는 참가자가 아무나 열 수 있음.
    4. 시작시간 10분 이후에는 세션이 취소된 상태로 간주하고 아무도 열 수 없음.
  • 구현 사항

    • validator에 대기방 생성에 대한 검증 로직 추가
    • 대기방을 생성한 사람은 id를 저장. >> 추후 통계 작업 시 대기방의 방장의 기록을 사용
    • 방장에게 passcode와 run_public_id 반환

트러블 슈팅

리뷰어에게 하고 싶은 말을 적어주세요

  • 노션의 비즈니스 요구사항을 열심히 보고 개발했는데, 혹시 잘못된 로직이 있다면 꼭 피드백 부탁드립니다!

확인하기

  • : 코드에 에러가 없는지 확인했나요?
  • : PR에 설명을 기재했나요?
  • : PR 태그를 붙였나요?
  • : 불필요한 로그나 System.out을 제거했나요?

@ekgns33 ekgns33 requested review from hughesgoon and jeeheaG January 11, 2025 10:01
@ekgns33 ekgns33 self-assigned this Jan 11, 2025
@hughesgoon
Copy link
Collaborator

ㅓㅇ으ㅡ악 지금 구현중이라 코드 리뷰가 늦어져 죄송합니다.
일단 말씀주신 Security는 이번 PR에 Resolver고치는거랑 포함해서 한번에 올리겠습니다.

@hughesgoon
Copy link
Collaborator

hughesgoon commented Jan 13, 2025

리뷰에 앞서,,, 컨벤션이 자주 바뀌어서 사실 맞추고 따라가는데 많은 어려움을 겪고있는데요.
controller가 반환하는 객체는 ResponseEntity로 알고있는데, 마이페이지는 SuccessResponse를 바로 반환하더라고요?
또, SuccessResponse가 이미 HTTP Status를 지니고 있는데, ResponseEntity에서도 하드코딩해서 HTTP Status를 한번 더 넣는건지요..?

@ekgns33
Copy link
Collaborator Author

ekgns33 commented Jan 13, 2025

controller가 반환하는 객체는 ResponseEntity로 알고있는데, 마이페이지는 SuccessResponse를 바로 반환하더라고요?

@hughesgoon
Run말고는 이번 작업 내용이 아니라 반영이 안된것 같습니다. 마지막 merge가 12/2이고 작업은 11월에 했네유 스펙 정하기로하고 땅땅땅한날이랑 달라서 반영이 안된것같습니다 😢

@ekgns33 ekgns33 merged commit 9b8291a into main Jan 16, 2025
@hughesgoon hughesgoon deleted the JIS-99-open-live-session branch January 23, 2025 06:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants