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 관리자 공고 등록 수정 템플릿 #1256 #1379

Merged

Conversation

greatSweetMango
Copy link
Collaborator

@greatSweetMango greatSweetMango commented Apr 8, 2024

📌 개요

API 연동 작업

💻 작업사항

  • recruitmentDtoList -> recruitments 명세에 맞추어 변경했습니다
  • 공고 불러오기에 드롭다운에 페이지네이션동작 추가했습니다
  • 공고 상태 수정 API 요청
  • 새 질문지 항목 추가시 스크롤 자동이동
  • 필요없는 로그, 주석, import 제거

TODO :

  • 코드 리펙터링

✅ 변경로직

  • 기존 명세에서 내용이 바뀌어 타입 수정했습니다. (form -> forms, content -> contens)

@greatSweetMango greatSweetMango added the enhancement New feature or request label Apr 8, 2024
@greatSweetMango greatSweetMango self-assigned this Apr 8, 2024
@PHJoon
Copy link
Contributor

PHJoon commented Apr 9, 2024

QuestionFormBuilder와 QuestionForm의 SingleCheck, MultiCheck 컴포넌트의 value를 content -> contents로 변경해야 해당 선택사항 내용이 보이는 것 같아요
상세보기 페이지에서는 label이 안보이면 좋을 것 같아요
그리고 수정페이지에서 불러오는 SingleCheck와 MultiCheck 컴포넌트에서는 value대신 defaultValue 쓰거나 state로 관리해야 수정이 가능할 것 같아요.
상세보기와 수정하기 구분이 확실히 필요하다면 상세보기에서는 질문과 선택사항들에 다 disabled 걸어도 될 듯 합니다

@@ -198,7 +198,7 @@ export default function useRecruitmentEditInfo(
const importRecruitmentInfo = async (recruitId: number) => {
if (recruitId <= 0) return;
try {
const res = await instance.get('/recruitments/' + recruitId);
const res = await instance.get('/admin/recruitments/' + recruitId);
const data: Irecruit = {
Copy link
Contributor

@PHJoon PHJoon Apr 10, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

form: res.data.forms로 바꿔야 할 것 같아요(208번째 줄)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

title: res.data.title,
startDate: new Date(res.data.startDate),
endDate: new Date(res.data.endDate),
generation: res.data.generation,
contents: res.data.contents,
form: res.data.form,
form: res.data.forms,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이것도 forms로 타입 통일해주는게 좋지 않을까요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

API 명세가 생성 수정에서는 form 조회시에는 forms여서 api 명세 통일했습니다.

Copy link
Member

@hyobb109 hyobb109 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저도 상세보기에서는 질문 창에 입력 막아두고 label 지우는 게 좋을 것 같습니다. 그리고 객관식 보기가 안보이는 것도 수정이 필요할 것 같아요. 등록이나 삭제는 잘 되는 것 같습니다!

Copy link
Member

@yoouyeon yoouyeon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인이 늦어 죄송합니다... 작업하신 내용 모두 잘 동작하는 것 같습니다! 오타 하나만 수정해주세요 고생하셨습니다.👏

// contents: recruitmentEditInfo.contents,
// form: recruitmentEditInfo.form,
// });
alert('공고를 성공적을 생성하였습니다.');
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오타 수정해주세여!

Copy link
Contributor

@PHJoon PHJoon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다!! 👍

@greatSweetMango greatSweetMango merged commit 748502f into recruit Apr 15, 2024
@greatSweetMango greatSweetMango deleted the feat-관리자-공고-등록-수정-템플릿-#1256 branch April 15, 2024 05:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants