인증목록 api 내 페이징 처리 로직 수정

[Hwangseoeun]

😉 연관 이슈

Resolves #233

🧑‍💻 수행 작업

기존에는 단순히 페이징 처리 시 DB에 들어있는 데이터 순으로 조회한 다음 페이징을 적용하는 방식이었습니다. 이러한 문제로 인해 더미 데이터를 추가하거나 인증 투두에 문제가 생겼을 때 조회가 이상하게 되는 문제가 발생하여 이를 수정하였습니다.
수정한 결과 DB에 들어있는 데이터들을 우선 인증 날짜를 기준으로 내림차순 정렬을 한 뒤 페이징 처리를 적용하도록 하였습니다.

📢 참고 사항

X

Summary by CodeRabbit

릴리스 노트

• Bug Fixes

  • 인증 데이터 조회 시 최신순(생성 시간 기준 내림차순) 정렬이 일관되게 적용되었습니다.

• Chores

  • 로컬 환경 데이터베이스 스키마 초기화 방식을 업데이트했습니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

저장소 인터페이스의 쿼리 메서드 시그니처를 업데이트하여 생성 시간 기준 내림차순 정렬을 강제하였습니다. 서비스 계층에서 새로운 정렬 메서드를 사용하도록 수정하였으며, 로컬 환경 설정에서 하이버네이트 DDL 전략을 변경하였습니다.

Changes

Cohort / File(s)	요약
저장소 쿼리 메서드 정렬 추가 src/main/java/site/dogether/dailytodocertification/repository/DailyTodoCertificationRepository.java	네 개의 쿼리 메서드 시그니처 변경: findAllByDailyTodo_Member() → findAllByDailyTodo_MemberOrderByCreatedAtDesc(), findAllByDailyTodo_MemberAndReviewStatus() → findAllByDailyTodo_MemberAndReviewStatusOrderByCreatedAtDesc(). 리스트 및 슬라이스 기반 메서드 모두 포함.
서비스 계층 저장소 호출 업데이트 src/main/java/site/dogether/memberactivity/service/MemberActivityService.java, src/main/java/site/dogether/memberactivity/service/MemberActivityServiceV1.java	새로운 정렬 메서드 이름을 사용하도록 저장소 호출 수정. 상태 필터 여부에 따라 OrderByCreatedAtDesc 메서드 선택. MemberActivityServiceV1에서는 추가로 from(certificationsBySlice) 호출 확대.
설정 파일 수정 src/main/resources/application-local.yml	spring.jpa.hibernate.ddl-auto 값을 create에서 update로 변경하여 스키마 재생성 대신 업데이트 수행.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• 저장소 인터페이스의 일관된 메서드 이름 변경 패턴이 반복되므로 이해도는 높으나, 서비스 계층의 다중 호출 지점 확인 필요
• MemberActivityServiceV1의 from(certificationsBySlice) 추가 호출이 응답 구성에 미치는 영향 검증 필요
• 정렬 순서 변경이 페이징 처리 시 원하는 결과를 제공하는지 확인 필요

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/#233-member_certifications_paging_refactoring

---

📜 Recent review details

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between facb5ee and 4b67d9e.

📒 Files selected for processing (4)

• src/main/java/site/dogether/dailytodocertification/repository/DailyTodoCertificationRepository.java (1 hunks)
• src/main/java/site/dogether/memberactivity/service/MemberActivityService.java (1 hunks)
• src/main/java/site/dogether/memberactivity/service/MemberActivityServiceV1.java (2 hunks)
• src/main/resources/application-local.yml (1 hunks)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}