Skip to content

[새기능] 원서에 페이지네이션 적용 - #390

Open
somonox wants to merge 12 commits into
developfrom
refactor/#389
Open

[새기능] 원서에 페이지네이션 적용#390
somonox wants to merge 12 commits into
developfrom
refactor/#389

Conversation

@somonox

@somonox somonox commented May 6, 2026 •
edited by leegunwoooo
Loading

Copy link
Copy Markdown

🎫 관련 이슈

close #


📄 개요


🔨 작업 내용


🏁 확인 사항

  • 테스트를 완료했나요?
  • API 문서를 작성했나요?
  • 코드 컨벤션을 준수했나요?
  • 불필요한 로그, 주석, import 등을 삭제했나요?

🙋🏻 덧붙일 말

somonox added 8 commits May 6, 2026 12:08
@somonox
refactor(QueryAllFormUseCase.java): overload execute method for pagin…
fe92a06 
…ation support

AdminFormController.java에서 getFormList의 패이지네이션 지원을 위해 기존 execute를 오버로드한 새 함수를 만듦.
@somonox
refactor(AdminFormController.java): add pagination support to form query
8f6d12a 
명확히 하위 호환성을 준수하도록하였고 페이지네이션 파라미터가 부분적이더라도 기본값으로 처리되도록 하였습니다.
@somonox
test(#389): add test for partial form retrieval with pagination
a93adfc 
전체, 일부 모두 성공적으로 작동함을 확인하였습니다.
@somonox
test(#389): add test for partial form retrieval with pagination
844ac9f 
전체, 일부 모두 성공적으로 작동함을 확인하였습니다.
@somonox
test(#389): add test for partial form retrieval with pagination
14a6932
@somonox
refactor(PageResult.java): create PageResult class for pagination res…
9b27f97 
…ponse
@somonox
refactor(AdminFormController.java): update getFormList method to retu…
fddabd3 
…rn paginated response

DTO의 일관성을 유지하기 위해 DTO를 수정하는 대신 헤더로써 메타데이터를 반환토록 하였습니다.
@somonox
refactor(QueryAllFormUseCase.java): update execute method to return p…
e64dbf0 
…aginated results
@somonox
somonox requested a review from a team as a code owner May 6, 2026 06:59
@github-actions

github-actions Bot commented May 6, 2026

Copy link
Copy Markdown

이렇게 리뷰해주세요.

🖐️ 규칙

  1. 왜 개선이 필요한지 충분히 설명해주세요.
  2. 답을 알려주기보다는 스스로 고민하고 개선 방법을 선택할 수 있게 해주세요.
  3. 코드를 깔끔하게 유지하고, 일관되게 구현하도록 안내해 주세요.
  4. 리뷰 과정이 숙제검사가 아닌 학습과정으로 느낄 수 있게 리뷰해 주세요.
  5. 리뷰를 위한 리뷰를 하지 마세요. 피드백 할 게 없으면 칭찬해 주세요.

🥬 우선순위

리뷰 중요도에 따라 다음과 같은 prefix를 붙여주세요.

예시 - P5) 쓸모없는 주석 삭제해주세요.

  • P1: 꼭 반영해주세요 (Request changes)
  • P2: 적극적으로 고려해주세요 (Request changes)
  • P3: 웬만하면 반영해 주세요 (Comment)
  • P4: 반영해도 좋고 넘어가도 좋습니다 (Approve)
  • P5: 그냥 사소한 의견입니다 (Approve)

@somonox somonox changed the title [개선] 원서 조회의 페이지네이션 [새기능] 원서에 페이지네이션 적용 May 6, 2026
leegunwoooo
leegunwoooo requested changes May 6, 2026
View reviewed changes
Comment thread src/main/java/com/bamdoliro/maru/application/form/QueryAllFormUseCase.java Outdated
Comment thread src/main/java/com/bamdoliro/maru/presentation/form/AdminFormController.java
Comment thread src/main/java/com/bamdoliro/maru/presentation/form/AdminFormController.java
Comment thread src/main/java/com/bamdoliro/maru/application/form/QueryAllFormUseCase.java Outdated
Comment thread src/main/java/com/bamdoliro/maru/presentation/form/AdminFormController.java Outdated
@leegunwoooo

leegunwoooo commented May 6, 2026

Copy link
Copy Markdown
Member

P1) 주석제거해주세용용

leegunwoooo
leegunwoooo requested changes May 7, 2026
View reviewed changes
Comment thread src/main/java/com/bamdoliro/maru/presentation/form/AdminFormController.java Outdated
@sonarqubecloud

sonarqubecloud Bot commented May 9, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@leegunwoooo

leegunwoooo commented May 10, 2026

Copy link
Copy Markdown
Member

수고하셨습니다! 사실 마루라는 프로젝트 특성상 페이지네이션이 필요하지 않아요. 제가 해당 이슈를 통해 시킨 이유는 5기 밤돌이로 분들이 마루 코드에 대한 감과 컨벤션에 익숙해졌으면 함이었습니다. 수고하셨어요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@leegunwoooo leegunwoooo leegunwoooo approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@somonox @leegunwoooo