Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 모집글 작성, 모집글 수정 페이지 쿼리 셋팅 수정 및 관련 타입 오류 해결 - post api 관련 글 작성하기, 수정하기 함수 타입변경 - 드랍다운 관련 상수에 각 value 추가
- handleSubmit 함수에 모집글 작성 유효성 처리 기능 추가 (요청 처리 오류시 알림창, 스크롤 최상단으로 이동)
올바른 PR 형식올바른 PR 형식입니다. |
JIS0098
requested review from
kimbobby,
nakyoung98,
hankim0904 and
Yoon-kyungLee
hankim0904
approved these changes
Mar 25, 2024
kimbobby
approved these changes
Mar 25, 2024
nakyoung98
approved these changes
Mar 25, 2024
| progressWay: "", | ||
| stacks: [], | ||
| progressPeriod: null, | ||
| capacity: 10, |
There was a problem hiding this comment.
모집인원 관련 드랍다운 버튼에 선택을 아예 안했을 때와 인원 미정을 선택한 값이 null로 같아서
지금은 임시로 설정해 두었습니다 드랍다운 내부 수정해 놓겠습니다!-!
|
|
||
| const findOptionByValue = <ValueType, OptionType>(values: ValueType[], options: OptionType[], value: ValueType) => { | ||
| const index = values.indexOf(value); | ||
| return options[index]; | ||
| }; | ||
|
|
||
| const handleSelectType = (type: string): void => { | ||
| setSelectedOption((prevOptions) => ({ | ||
| const handleSelectType = (type: "STUDY" | "PROJECT"): void => { |
| @@ -1,5 +1,5 @@ | |||
| export interface RecruitmentRequest { | |||
| type: string; | |||
| type: "STUDY" | "PROJECT"; | |||
There was a problem hiding this comment.
이부분.. 못보고 머지했네요ㅠ 다음에 pr올릴때 꼭 수정해서 올리겠습니다!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📌요구사항
📌작업 진행 상황 (에러, 막혔던 부분, 그외 궁금한것 등등)
이번에 작업하면서(쿼리셋팅, 요청처리)기술스택 부분에 오류가 있어서 나경님이랑 수정 해야할 것 같아
일단 주석 처리 하고 올리겠습니다
지금 리팩토링(리액트훅폼 컨트롤+드랍다운)을 하면 파일을 계속 옮겨 다니면서 작업해야 해서
api연동까지만 확인 되면 리팩토링 해 놓겠습니다 ㅠ
지금 constants/dropDowns에 드랍버튼의 text가 options에 있는 값이 아닌 다른 값으로 보여져야
하는 곳이 모집글 작성, 수정 페이지 외에도 생길 수 있어서 일단 value 다 추가해 놨습니다 기능구현 하시다
비슷한 문제가 발생하면 value를 참고해서 사용하시면 될 것 같습니다
링크 유효성 검사를 통과 못하면 글 작성이 안되는 걸까요..?-? 현재는 링크 입력창은 요청 조건에 넣진 않았습니다
필수 입력값을 안넣으면 또 스크롤해서 올라가는게 불편해서 알림창 띄우고 최상단으로 스크롤 가게 해놨는데
괜찮은지 확인 부탁 드립니다!!
📌스크린샷 / 테스트결과 (선택)
📌이슈 번호
close #147