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: 모집글 페이지 드랍다운 관련 기능 수정 #137

Conversation

JIS0098
Copy link
Contributor

@JIS0098 JIS0098 commented Mar 22, 2024

📌요구사항

  • 모집글 작성 페이지 관련 드랍다운 내용 변경
  • 드랍다운 필수조건 추가
  • 링크 관련 입력창 추가(+입력창 유효성 검사)

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

  • 일단 기능구현 먼저하느라 코드를 한파일에 다 적어놨는데 내일 노트북으로 작업을 할 예정이라
    일단 피알 먼저 올리고 주말에 리팩토링이랑 수정할 부분 이어하겠습니다

  • 하은님이 작성해주신 유효성 검사 상수를 적용 시키려했는데 타입 오류가 나서
    일단 정규식 하드 코딩했습니다 내일 위워크 오시면 여쭤보고 수정하겠습니다

  • 링크 인풋에 이메일은 검사가 잘되는데 url관련 검사 부분에서 유효성 검사가 제대로 안되고있어
    리팩토링 하면서 같이 수정해야할 것 같습니다

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

드랍다운, 인풋 유효성

📌이슈 번호

close #135

Copy link

올바른 PR 형식

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

@@ -2,7 +2,7 @@ export interface RecruitmentRequest {
type: string;
recruitEndAt: string;
progressPeriod: string;
capacity: number;
capacity: number | undefined | null;
Copy link
Contributor

Choose a reason for hiding this comment

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

백엔드 분들이랑 논의 부탁드려용

@@ -17,20 +17,21 @@ export const DROPDOWN_INFO = {
recruitment: {
progressPeriod: {
defaultValue: "진행 기간",
options: ["1주", "2주", "3주", "1개월", "2개월", "3개월", "4개월", "5개월", "6개월", "6개월 이상"],
options: ["선택 미정", "1주", "2주", "3주", "1개월", "2개월", "3개월", "4개월", "5개월", "6개월", "6개월 이상"],
Copy link
Contributor

Choose a reason for hiding this comment

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

진행기간 "선택 미정" => 협의 후 결정
모집인원 "선택 미정" => 인원 미정
연락 방법 "선택 미정" => 기타
로 변경 해주세요

@hankim0904 hankim0904 merged commit b14e96d into develop Mar 23, 2024
1 check passed
@hankim0904 hankim0904 deleted the feature/issue-135--Feat-모집글-페이지-드랍다운-관련-기능-수정 branch March 23, 2024 02:29
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] :모집글 페이지 드랍다운 관련 기능 수정
4 participants