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

Feat: RecruitmentRequestLayout 수정 및 Recruit, Edit 페이지 api 연동 #178

Merged
merged 7 commits into from
Mar 27, 2024

Conversation

JIS0098
Copy link
Contributor

@JIS0098 JIS0098 commented Mar 27, 2024

📌요구사항

  • RecruitmentRequestLayout.tsx 모집포지션 칩 관련 코드 수정
  • 칩을 처음 선택한 뒤 재클릭 후(선택취소)에도 값이 있는걸로 인식됨,
  • Request 조건에 인풋창 유효성 검사 추가)
  • Recruit, Edit 페이지 api 연동
  • api 연동 확인되면 리팩토링

📌작업 진행 상황 (에러, 막혔던 부분, 그외 궁금한것 등등)

  • 어제 Edit 페이지도 api 요청 처리는 성공했는데 Edit 페이지에 공통레이이아웃을 쓰면
    해당 포스트의 value값이 렌더링되서 바껴 보여도 훅 폼으로 유효성 검사를 건너뛰어서
    필수조건을 다시 선택 안하면 훅 폼 에러 메세지가 뜹니다 다음 작업 때 이부분 수정하고
    Edit 페이지 제작할 것 같습니다
  • RecruitmentRequestLayout 드랍다운 관련 리팩토링을 시도했는데 속성이 조금씩 다 달라서..
    가능한 부분과 util 함수 정도 분리했습니다
interface RecruitmentRequestLayoutProps {
  mutationFn: any; // 교체예정입니다
  buttonText: string;
  selectedOptions: RecruitApiRequestDto;
  setSelectedOptions: React.Dispatch<React.SetStateAction<RecruitApiRequestDto>>;
}

이 부분 방금 리팩토링 하면서 mutationFn로 수정됐는데 타입이 바로 생각이 안나서
일단 any 로 놨습니다 ㅠ 추후에 수정해 놓겠습니다

📌스크린샷 / 테스트결과 (선택)

모집글 작성하기

📌이슈 번호

close #161

@JIS0098 JIS0098 self-assigned this Mar 27, 2024
Copy link

올바른 PR 형식

올바른 PR 형식입니다.
코드 작성하시느라 수고 많으셨습니다 :)

@JIS0098 JIS0098 merged commit 2882b52 into develop Mar 27, 2024
1 check passed
@JIS0098 JIS0098 deleted the feature/issue-161--Feat-RecruitmentRequestLayou branch March 28, 2024 19:19
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.

[Feat] : RecruitmentRequestLayout 수정 및 Recruit, Edit 페이지 api 연동
5 participants