Skip to content

refactor: 전체 채널 조회 API 쿼리 파라미터 미지정 시 전체 반환 - #113

Open
1117mg wants to merge 1 commit into
devfrom
feat/105-channel-list-unpaged-by-default
Open

refactor: 전체 채널 조회 API 쿼리 파라미터 미지정 시 전체 반환#113
1117mg wants to merge 1 commit into
devfrom
feat/105-channel-list-unpaged-by-default

Conversation

@1117mg

@1117mg 1117mg commented Jul 31, 2026

Copy link
Copy Markdown
Member

🔗 관련 이슈

Closes #105

📝 작업 내용

  • GET /api/v1/channels 에서 page/size 를 모두 생략하면 페이지네이션 없이 활성 채널 전체를 반환하도록 변경 (기존: 이름순 12개)
  • page 또는 size 중 하나라도 지정하면 페이지 조회로 동작하고, 생략된 값은 page=0 / size=12 로 채움

요청 예시

  • GET /channels : 활성 채널 전체, 이름순
  • GET /channels?page=0&size=12 : 0페이지 12개
  • GET /channels?size=20 : 0페이지 20개
  • GET /channels?page=2 : 2페이지, size 기본 12
  • GET /channels?name=11번가 : 검색 결과 전체
  • GET /channels?name=11번가&size=12 : 검색 결과 0페이지 12개
  • GET /channels?size=0 : 400 C-001

✅ 체크리스트

  • 로컬에서 빌드 및 테스트가 통과했습니다.
  • 컨벤션(브랜치/커밋 메시지)을 준수했습니다.
  • 관련 문서를 수정했습니다. (필요한 경우)

@1117mg 1117mg self-assigned this Jul 31, 2026
@1117mg
1117mg changed the base branch from main to dev July 31, 2026 08:32
@1117mg 1117mg closed this Jul 31, 2026
@1117mg 1117mg reopened this Jul 31, 2026
@1117mg
1117mg requested a review from SinnoLn July 31, 2026 08:32
@claude

claude Bot commented Jul 31, 2026

Copy link
Copy Markdown

PR 리뷰 결과 (자동 머지 게이트)

머지를 막을 만한 필수 수정 사항을 찾지 못했습니다.

검토 범위 및 확인 내용

  • ChannelRepositoryImpl: pageable.isPaged() 분기로 unpaged 조회 시 offset/limit 생략 — PageableExecutionUtils.getPage는 unpaged일 때 count 쿼리를 실행하지 않으므로 성능상 문제 없음
  • ChannelSearchRequest.toPageable: page/size 둘 다 미지정 시 Pageable.unpaged(sort), 하나라도 지정 시 page=0/size=12 기본값 적용 및 size 1~100 검증 — PR 설명과 일치, 기존 CommonErrorCode.INVALID_INPUT_VALUE 패턴(동일 파일 내 toOrderSpecifiers)과 일관됨
  • ArchUnit 계층 규칙: ChannelSearchRequest*Request 네이밍으로 presentation.dto 패키지에 위치 — 규칙 위반 없음
  • ChannelController/ChannelApiDocs@ParameterObject ChannelSearchRequest + @SortDefault Sort 조합, PK/엔티티/Flyway 변경 없음 — 특이사항 없음
  • 테스트(ChannelRepositoryTest, ChannelControllerTest)가 unpaged/paged/size 경계값(0, 100, 101)을 모두 커버

Blocker/Major/Minor 레벨에서 보고할 항목이 없습니다.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] 전체 채널 조회 API에서 쿼리 파라미터 미지정 시 전체 채널 반환

2 participants