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

로드맵 서비스 가독성 개선 #1617

Closed
wants to merge 21 commits into from
Closed

Conversation

nuyh99
Copy link
Contributor

@nuyh99 nuyh99 commented Nov 22, 2023

#️⃣연관된 이슈

📝작업 내용

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

기존 요구 사항

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

바뀐 요구 사항

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

java-saeng and others added 21 commits October 11, 2023 16:50
게스트 회원은 로드맵 답변을 쓸 수 없다
article develop, develop 브랜치에 머지
hotfix: 로그인 시 프로필 표시 하도록 수정
fix: 아이콘 Dom 구조 변경으로 인한 드롭다운 표시 에러 수정
…ess-not-showing-on-safari

Safari에서 로드맵 퀴즈 progress가 표시되지 않는 오류를 수정
Member, GroupMember, MemberGroup 엔티티 재설계
@nuyh99 nuyh99 closed this Nov 22, 2023
Copy link

sonarcloud bot commented Nov 22, 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 2 Code Smells

94.7% 94.7% 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

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.

7 participants