Skip to content

Conversation

@parkjaehak
Copy link
Collaborator

@parkjaehak parkjaehak commented Jan 25, 2025

📄 요약(Summary)

작업 상태가 REQUESTED인 상태에 대한 모든 작업 목록 회

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • secured 애노테이션을 통해 manager 만 접근하도록 설정
  • taskStatus가 REQUESTED인 상태에 대해 모든 작업 목록 조회
  • 필터링, 정렬, 페이징 기능 사용자 요청 목록 조회 로직 재사용

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

  • 서비스 단 단위테스트만을 작성했는데 사실상 필터링, 정렬 기능에 대한 테스트를 할 수 없어 무의미하지 않나 생각이 듭니다.
  • 추후 통합 테스트를 작성하게되면 그때 필터링, 정렬에 대한 기능이 작동하는지 테스트해보아야할 것 같습니다. (postman 실행시 주어진 값에 따라 필터링, 정렬에 대한 테스트는 된 상태이지만 모든 케이스에 대해 테스트 케이스 작성을 통해 문서화를 해놓는게 좋을 것 같습니다.)

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

첨부 자료

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

✅ 체크리스트(Checklist)

  • PR 양식에 맞게 작성했습니다
  • 모든 테스트가 통과했습니다
  • 프로그램이 정상적으로 작동합니다
  • 적절한 PR 라벨을 설정했습니다
  • 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #17

@parkjaehak parkjaehak added ✨ feature 구현·개선 사항에 관련된 내용입니다 HIGH 우선순위 상 labels Jan 25, 2025
@parkjaehak parkjaehak self-assigned this Jan 25, 2025
@parkjaehak parkjaehak merged commit 9082009 into develop Jan 25, 2025
1 check passed
@joowojr joowojr deleted the CLAP-59 branch January 26, 2025 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feature 구현·개선 사항에 관련된 내용입니다 HIGH 우선순위 상

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLAP-59 승인 대기 중인 요청 목록 조회 API

3 participants