Conversation
added 2 commits
July 13, 2023 17:12
- 한글과 영어로만 구성되는 닉네임 규칙에따라 제한 - 정규 표현식으로 한글, 영어가 아닌 문자는 빈문자로 대체시킴 = 테스트 결과 개행이 들어갈 수있어서 추가로 개행도 대체 표현식에 추가시킴
- 유저 닉네임을 제외한 모든 input 값에 대해 앞 뒤 공백제거 적용 - onChange에 useInput 훅의 onHandler 를 사용하는 곳 전부에 적용 시키조가 useInput 변경 앞 공백 제거 - 아이템 추가/수정 컴포넌트에서 아이템 닉네임, 브랜드 input에서 onBlur시 뒷 공백 제거 - 사용자가 엔터를 치는 경우에도 같은 동작을 하기위해 onsubmit 시 trim된 값 서버에 전달
sumnii
reviewed
Jul 21, 2023
sumnii
reviewed
Jul 21, 2023
Collaborator
sumnii
left a comment
sumnii
left a comment
There was a problem hiding this comment.
확인했습니다..! 생각보다 문서가 깐깐하더라구요..그런데도 잘 구현하셨어용 👍
length 오타 있는 부분 왜 돌아가는지 이유를 모를...저 부분에 걸리지 않는 건가..? 좀 지켜봐야겠어요
아이템 재질 not null 문제가 있어서 대호랑 조율중입니다
src/pages/item/components/modification/ItemEdit/ItemEditDetail.jsx
Outdated
Show resolved
Hide resolved
Co-authored-by: Sumin Song <sumsong@student.42seoul.kr>
….jsx Co-authored-by: Sumin Song <sumsong@student.42seoul.kr>
- 한 가지 입력값만이 아니라 모든 양식 데이터에대한 검증의 결과이므로 일단 잘못된 양식을 제출했다는 안내 제공
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
개요
작업사항
정규표현식을 이용해서 구햔
ux고려해서 입력중엔 제어안함
onblur onsubmit시에 제어