Skip to content

Feature/14 get issues api#20

Merged
meraki6512 merged 3 commits into
IssueDiver:devfrom
meraki6512:feature/14-get-issues-api
Aug 27, 2025
Merged

Feature/14 get issues api#20
meraki6512 merged 3 commits into
IssueDiver:devfrom
meraki6512:feature/14-get-issues-api

Conversation

@meraki6512
Copy link
Copy Markdown
Contributor

연관된 이슈

#14


작업 내용

  • 이슈 필터링 기능 구현
  • 테스트 환경 구축
    (페이징과 정렬 기능에 대한 단위 테스트 추가, 통합 테스트 환경 설정, Flyway 비활성화로 인한 테스트 DB 스키마 충돌 문제 해결, 그리고 테스트 보안 설정(@WithMockUser) 적용)

-> 이슈 필터링 기능의 DB 연동 동작을 검증하고, 안정적인 테스트 환경을 마련함.

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).
  • main branch가 아닌 dev branch에 PR 요청을 했습니다. (main branch에 바로 PR&merge하지 않기).

- 상태(status), 작성자(authorId), 라벨(labelIds) 등 동적 필터링 조건 적용한 IssueService 메서드 추가
- 검증된 IssueFilterRequest 파라미터를 받는 GET /issues REST API 컨트롤러 구현
- 페이징 처리된 IssueResponse를 공통 응답 포맷(ApiResponse)으로 감싸 반환
- 필터링 구현으로 수정된 엔티티에 따라 기존 테스트 코드 일부 수정
- 단위(페이징 및 정렬 기능) 테스트 코드 추가 및 검증
- 통합 테스트 환경 구성 및 연관 테스트 코드 추가 및 검증
- Flyway 비활성화로 테스트 DB 스키마 문제 해결
- 보안 설정(@WithMockUser) 적용
@meraki6512
Copy link
Copy Markdown
Contributor Author

meraki6512 commented Aug 27, 2025

Label entity 이슈 리스트 조회에서 필요해서 만들었는데, 기존 Label 코드와 다른 부분이 있어 수정 후 머지했습니다.
(팀 코드 컨벤션에 따라서 카멜 케이스로 변경)
@tpgus1221

@meraki6512 meraki6512 merged commit d7db05a into IssueDiver:dev Aug 27, 2025
@meraki6512 meraki6512 deleted the feature/14-get-issues-api branch August 27, 2025 17:50
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.

1 participant