Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fix] recruit admin api 연결 중 발생한 에러 수정 (조회 api 위주) #1360

Conversation

yoouyeon
Copy link
Member

@yoouyeon yoouyeon commented Apr 6, 2024

📌 개요

  • recruit admin api 테스트 중 발견된 에러를 수정했습니다.

💻 작업사항

전반적 변경사항

임의로 수정한 부분이나 보완해야 할 부분 FIXME 주석으로 남겨두었으니 확인후 수정 부탁드립니다. @parksangmin1543 @greatSweetMango

  • axios instance 변경
    instanceInManage -> instance (url에서 pingpong이 제거됨)
  • 데이터 프로퍼티명 변경 => 프로퍼티명 논의할 시간이 없어서 일단 구현된 api 명세에 맞췄습니다.
    • recruitments -> recruitmentDtoList
    • applications -> applicationResults

useRecruitmentUserFilter

  • 훅 이름 변경
    useRucruitmentUserFilter -> useRecruitmentUserFilter
  • recruitId 필수로 변경 (undefined일 경우에는 요청 자체가 불가능함)
  • 페이지네이션 기능 주석처리 (api는 없는 쪽으로 구현되어 있으므로 다시 논의 필요합니다)

FilterQptionsUI

  • recruitId 필수로 변경 (undefined일 경우에는 요청 자체가 불가능함)

DetailRecruitUserList

  • applicationResults undefined에러 처리 추가

✅ 변경로직

yoouyeon added 9 commits April 6, 2024 16:33
- axios 인스턴스 변경
- axios 인스턴스 변경
- recruitments -> recruitmentDtoList
- recruitId 필수로 변경
- 페이지네이션 주석 처리
- 데이터 프로퍼티명 변경 (applications -> applicationResults)
- applicationResults undefined에러 처리 추가
- axios instance 변경
@yoouyeon yoouyeon added admin 관리자 페이지에서 발생한 issue recruit 모집 기능 라벨 labels Apr 6, 2024
@yoouyeon yoouyeon self-assigned this Apr 6, 2024
@yoouyeon
Copy link
Member Author

yoouyeon commented Apr 6, 2024

리뷰 가능 인원 부족으로 강제머지 하겠습니다.

@yoouyeon yoouyeon merged commit b1b2438 into recruit Apr 6, 2024
@yoouyeon yoouyeon deleted the fix/1359-fix-recruit-admin-api-연결-중-발견된-에러-수정 branch April 6, 2024 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
admin 관리자 페이지에서 발생한 issue recruit 모집 기능 라벨
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] recruit admin api 연결 중 발견된 에러 수정 (조회 api 위주)
1 participant