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.

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

Feat : 마이페이지 기능을 위한 저장기능을 구현한다 #30

Merged
merged 5 commits into from
Jan 22, 2025

Conversation

chaen-ing
Copy link
Member

@chaen-ing chaen-ing commented Jan 21, 2025

#️⃣연관된 이슈

#25

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

스크린샷 2025-01-22 오전 3 20 51
  • 저장한 학습 조회를 위한 학습 저장 기능
  • 저장한 퀴즈 조회를 위한 퀴즈 저장 기능
스크린샷 2025-01-22 오전 3 22 29
  • 저장한 퀴즈를 다시 풀기 위해 개별 퀴즈 조회 구현
  • 퀴즈 제출 api에 개별 퀴즈 제출을 위한 로직 추가

💬리뷰 요구사항(선택)

퀴즈 제출 로직(QuizService)에서 redis에 데이터가 없는 경우 exception발생시키는 부분에 개별 퀴즈 제출 로직을 추가하였는데 괜찮을지 확인부탁드려요

learning 브랜치에 작업해서 먼저 pr보냅니다

@chaen-ing chaen-ing added the ✨ Feat 새로운 기능 추가 label Jan 21, 2025
@chaen-ing chaen-ing added this to the #3 - 마이페이지 1/2 milestone Jan 21, 2025
@chaen-ing chaen-ing requested a review from chanmin-00 January 21, 2025 18:23
@chaen-ing chaen-ing self-assigned this Jan 21, 2025
Copy link
Member

@chanmin-00 chanmin-00 left a comment

Choose a reason for hiding this comment

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

코드가 깔끔하니 좋네요 수고하셨습니다!

@PostMapping("/learning/quiz/{quizId}/scrap")
public ResponseEntity<ApiResponse<?>> scrapQuiz(
final @AuthenticationPrincipal CustomUserDetails currentUser,
final @PathVariable("quizId") long conceptId) {
Copy link
Member

Choose a reason for hiding this comment

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

여기 변수명 수정 필요할거 같아요

Copy link
Member Author

Choose a reason for hiding this comment

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

오 그러네요 수정하겠스빈다!!

@@ -83,17 +88,17 @@ public QuizResultDTO submitAnswer(final long userId, final long quizId, final in

QuizListDTO quizListDTO =
quizRedisService.fetchFromRedis(userId, QUESTION_TYPE, QuizListDTO.class);

// 마이페이지의 저장한 퀴즈에서 다시풀기 진행 시 (퀴즈 진행 redis 데이터가 없을 경우)
Copy link
Member

Choose a reason for hiding this comment

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

오 이 부분은 생각 못했는데 추가해주셔서 감사합니다

.answer()
.trim()
.equals(quizDTO.choiceList().choices().get(answerIndex).content().trim());
}
Copy link
Member

Choose a reason for hiding this comment

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

메소드로 따로 뺀 부분 좋은거 같아요

@chaen-ing chaen-ing merged commit ed4e253 into develop Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feat 새로운 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants