Skip to content

Conversation

@sunninz
Copy link
Contributor

@sunninz sunninz commented Sep 18, 2025

👀 관련 이슈

#314

✨ 작업한 내용

소켓 통신 이전에 강의에서 진행된 채팅 내용을 불러오기 위한 API 엔드포인트입니다.
Redis List에 저장된 채팅 내용을 불러오도록 구현했습니다.

🌀 PR Point

GET /api/lectures/chatting/before/{lectureId}

🍰 참고사항

테스트를 위해서 redis list에 저장된 채팅 목록이 있어야 합니다. 채팅 기능 구현 PR 머지 이후에 검토해주세요

📷 스크린샷 또는 GIF

X

@sunninz sunninz self-assigned this Sep 18, 2025
@sunninz sunninz linked an issue Sep 18, 2025 that may be closed by this pull request
1 task
Copy link
Contributor

@iinuyha iinuyha left a comment

Choose a reason for hiding this comment

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

혹시 내가 하면 403 Forbidden가 뜨는데 왤까나..ㅜㅜ

@AllArgsConstructor
public enum QuestionErrorCode implements BaseErrorCode {
_FORBIDDEN_LECTURE_ACCESS(HttpStatus.FORBIDDEN,"QUESTION403_1","해당 강의를 수강중인 학생만 조회가능합니다.");
_FORBIDDEN_LECTURE_ACCESS(HttpStatus.FORBIDDEN,"QUESTION403_1","해당 강의를 수강중인 학생만 조회가능합니다."),
Copy link
Contributor

Choose a reason for hiding this comment

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

나 자꾸 이 에러가 나는데 학생권한 관련 로직 다시 한 번 체크해줄 수 있을까용

Copy link
Contributor Author

Choose a reason for hiding this comment

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

GET api/lectures/chatting/before/{lectureId} 이 API 맞나용
위에 API에서는 학생검증 로직이 없어서용

Copy link
Contributor

Choose a reason for hiding this comment

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

image 웅 여기 403에러가떠서..ㅜㅡㅜ

Copy link
Contributor

Choose a reason for hiding this comment

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

미아내 해결함!!

@sunninz
Copy link
Contributor Author

sunninz commented Oct 5, 2025

혹시 내가 하면 403 Forbidden가 뜨는데 왤까나..ㅜㅜ

다시 풀당겨서 테스트 부탁드려용

Copy link
Contributor

@iinuyha iinuyha left a comment

Choose a reason for hiding this comment

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

프론트 연결 테스트 완료했습니다 👍

@sunninz
Copy link
Contributor Author

sunninz commented Oct 6, 2025

머지해도될까용

@sunninz sunninz merged commit fa8c57b into dev Oct 6, 2025
1 check passed
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.

✨ 이전 내용 불러오기 기능 구현

3 participants