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

로드맵 서비스 로직 리팩토링 #1618

Merged
merged 9 commits into from
Nov 23, 2023
Merged

로드맵 서비스 로직 리팩토링 #1618

merged 9 commits into from
Nov 23, 2023

Conversation

nuyh99
Copy link
Contributor

@nuyh99 nuyh99 commented Nov 23, 2023

#️⃣연관된 이슈

📝작업 내용

  • roadmap 패키지의 모든 Repository를 사용해서 자바 API를 통해 조립하던 기존 로직을 JPQL을 통해 개선했습니다.
    • 커리큘럼 ID에 해당하는 모든 키워드를 조회하는 findAllByCurriculumId() 구현
    • 키워드 ID 별로 총 퀴즈 개수를 조회하는 findTotalQuizCount() 구현
    • 키워드 ID 별로 현재 멤버가 답변한 총 퀴즈 개수를 조회하는 findDoneQuizCountByMemberId() 구현

기존 요구 사항

  • 세션 ID에 해당하는 키워드들을 한 번에 조회할 수 있다

바뀐 요구 사항

  • 커리큘럼 ID에 해당하는 모든 세션의 모든 키워드들을 한 번에 조회할 수 있다
    • 각 키워드별 총 퀴즈 개수도 같이 반환한다
    • 로그인한 유저가 조회하면 각 키워드별 답변한 총 퀴즈 개수도 같이 반환한다
    • 비로그인한 유저가 조회하면 각 키워드별 답변한 총 퀴즈 개수는 0개이다

@nuyh99 nuyh99 added BE 로드맵 로드맵 스쿼드 관련 이슈 labels Nov 23, 2023
@nuyh99 nuyh99 self-assigned this Nov 23, 2023
Copy link

sonarcloud bot commented Nov 23, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

94.4% 94.4% Coverage
0.0% 0.0% Duplication

warning The version of Java (11.0.21) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

@nuyh99 nuyh99 merged commit 7418118 into main Nov 23, 2023
2 checks passed
@nuyh99 nuyh99 deleted the refactor/1609 branch November 23, 2023 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 로드맵 로드맵 스쿼드 관련 이슈
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant