[기능구현] 보호자 시니어 추가 페이지 + 기타 코드 수정 #98
Conversation
Diwoni
added
✨ Feature
| @@ -44,6 +44,7 @@ const HeaderBox = styled.header` | |||
| justify-content: center; | |||
| align-items: center; | |||
| background-color: var(--color-white); | |||
| z-index: 999; | |||
There was a problem hiding this comment.
이런 css 코드도 있나요...?
https://developer.mozilla.org/ko/docs/Web/CSS/z-index
There was a problem hiding this comment.
999 값으로 설정한 건 그냥 막연히 헤더가 항상 다른 요소들을 가려야한다고 생각해서, 큰 정수 값을 넣었습니다..!
보기에 좀 그럴 수도 있겠네요.. 적당한 값 10 으로 수정할게요..ㅠ!
| export const formatPhoneNumber = (phone: string): string => { | ||
| const match = phone.match(/(\d{3})(\d{4})(\d{4})/); | ||
| return match ? `${match[1]}-${match[2]}-${match[3]}` : phone; | ||
| }; | ||
|
|
||
| export const parsePhoneNumber = (phone: string): string => { | ||
| const match = phone.match(/(\d{3})-(\d{4})-(\d{4})/); | ||
| return match ? `${match[1]}${match[2]}${match[3]}` : phone; | ||
| }; |
There was a problem hiding this comment.
이 코드가 utils 폴더에 위치한 이유가 있나요?? 저라면 useFormatPhoneNumber, useParsePhoneNumber 라고 정하고, shared/hooks 에 넣을 것 같아요.
There was a problem hiding this comment.
저도 shared/hooks 폴더에 처음에 넣었는데, 유틸리티 함수라고 보는 게 제 개인적인 생각에서는 개발하면서 사용하기에 유용한 함수(?) 느낌으로 바꿔 넣은 것 같아요. 둘 다 큰 상관 없을 것 같긴 합니다만 기존 hooks 폴더 안에 함수들과는 약간 다른 느낌에 옮겼습니다!
There was a problem hiding this comment.
저도 formatting 관련은 utils가 조금 더 적절한 것 같다고 생각합니다.
기존 Cascade가 순환참조 문제가 생겨서 해결했습니다 문제 생기면 곧바로 백엔드로 알려주시면 감사드리겠습니다 |
| <SeniorFormField | ||
| label='시니어의 전화번호' | ||
| placeholder='010-0000-0000' | ||
| error={errors.seniorPhoneNumber?.message} | ||
| registerProps={register('seniorPhoneNumber', { | ||
| required: '연락처를 입력해주세요.', | ||
| pattern: { | ||
| value: /^010-\d{4}-\d{4}$/, | ||
| message: '유효한 연락처 형식이 아닙니다.', | ||
| }, | ||
| })} | ||
| /> |
There was a problem hiding this comment.
제가 회원가입 ui 구현할 때 validation도 010-0000-0000 형태로만 입력할 수 있게 해놨긴 한데, 지금 생각해보니 시니어가 휴대전화가 아닌 집전화나 지역번호를 쓰는 경우도 있을까요? 만약 앞자리가 다른 번호도 사용 가능하게 한다면 validation을 약간 수정해야할 것 같아요. 아니면 010으로만 등록할 수 있도록 제한하고 수정하지 않아도 괜찮을 것 같습니다!
| export const formatPhoneNumber = (phone: string): string => { | ||
| const match = phone.match(/(\d{3})(\d{4})(\d{4})/); | ||
| return match ? `${match[1]}-${match[2]}-${match[3]}` : phone; | ||
| }; | ||
|
|
||
| export const parsePhoneNumber = (phone: string): string => { | ||
| const match = phone.match(/(\d{3})-(\d{4})-(\d{4})/); | ||
| return match ? `${match[1]}${match[2]}${match[3]}` : phone; | ||
| }; |
There was a problem hiding this comment.
저도 formatting 관련은 utils가 조금 더 적절한 것 같다고 생각합니다.
#️⃣ 연관된 이슈
📝 작업 내용
보호자 시니어 추가 페이지 기능 구현
전화번호 관련 hook
기타 코드 수정
스크린샷 (선택)
포인트 스타일 변경
시니어 추가 페이지
💬 리뷰 요구사항(선택)
⏰ 현재 버그
현재 시니어 추가 페이지에서 제가 시니어 추가 API 로 추가한 시니어는 삭제가 잘 되는데,
기존에 있던 시니어는 삭제가 되지 않는 문제(수정은 또 됨)가 있습니다.
콘솔로 확인했을 때도 시니어 추가 API 를 통해 추가한 시니어 객체와 기존에 db에 있던 시니어 정보들이 다르거나 별다르게 id 가 누락됐거나
하지 않은데 어디가 잘못된 건지 모르겠네요..ㅠ
✏ Git Close