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/#67] 펫 이름 뷰 구현 #68

Merged
merged 5 commits into from
Jan 17, 2025
Merged

[Feat/#67] 펫 이름 뷰 구현 #68

merged 5 commits into from
Jan 17, 2025

Conversation

yarimu
Copy link
Collaborator

@yarimu yarimu commented Jan 16, 2025

🔥 Related Issues

✅ 작업 리스트

  • 펫 이름 뷰 구현

🔧 작업 내용

펫 이름 뷰 구현 완료했습니다. 닉네임 등록 뷰에서 유효성 검사 로직만 수정한 정도여서 그쪽만 봐주시면 될 거 같아요!

  • 피그마 뷰
이름 등록뷰

구현한 뷰는 아래 동영상 첨부하겠습니다.

📣 리뷰어에게 어떠신가요?

progressbar는 온보딩 최상단에서 사용할 예정이라 지금 구현해두지 않았습니다!
🤔 궁금한점!! 아니 닉네임 뷰랑 똑같이 텍스트 필드 컴포넌트 불러다 썼는데 왜 갑자기 텍스트 필드에 x버튼이 생기는 걸까요?

📸 스크린샷 / GIF / Link

  • 구현 화면
2025-01-16.9.mp4

@yarimu yarimu added 🎨 design 사용자 UI 디자인 변경 - ex) CSS ✨ feat 기능 구현 labels Jan 16, 2025
@yarimu yarimu self-assigned this Jan 16, 2025
Copy link
Collaborator

@minjeoong minjeoong left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 코리한번만 달아주세요!!!!

Comment on lines +4 to +10
const errors: string[] = [];

// 영어 또는 숫자 하나라도 포함되면 오류
const alphanumericPattern = /[a-zA-Z0-9]/;
if (alphanumericPattern.test(petName)) {
errors.push(ERROR_MSG.petName.enNum); // 영어 또는 숫자가 포함된 경우
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

p4) error 라는 state 를 두고 setState 로 이전 상태 값에 에러를 추가하는 방법으로 세팅하는 방법이 있는데, 해당 방법으로 구현한 이유가 있을까용??

Copy link
Collaborator Author

@yarimu yarimu Jan 17, 2025

Choose a reason for hiding this comment

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

가장 큰 이유는 개인적인 취향이긴 한데, 상태 안두고 유틸 함수 작성하는 것을 좋아해서 입니도..호홍
그리고 해당 함수로 다른 컴포넌트에서 상태 비교를 통해 에러 메시지 렌더링도 해야하고, 버튼 활성화 유무도 결정해야해서 상태보다 함수로 들고다니는게 간편할 거라 생각했어요!

Copy link
Collaborator

Choose a reason for hiding this comment

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

p5) 아주 좋은 생각! 상태 많이 둬봤자 신경쓸 것만 많아져요~ 유틸함수 지향 잘 배웠네요! 좋습니다

Copy link
Collaborator

@ocahs9 ocahs9 left a comment

Choose a reason for hiding this comment

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

코멘트만 확인해주세요~
고생하셨습니다 ~!


export const btnWrapper = style({
maxWidth: "76.8rem",
width: "100%",
Copy link
Collaborator

Choose a reason for hiding this comment

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

p5) 미리 미리 고려해서 구현해두는 예림 누나 굿 ~

bottom: 0,

display: "grid",
gridTemplateColumns: "9.6rem calc(100% - 9.6rem)",
Copy link
Collaborator

Choose a reason for hiding this comment

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

p5) 이런 계산 식이라니.. 멋있다

Comment on lines +27 to +29
const handleGoBack = () => {
navigate(-1);
};
Copy link
Collaborator

Choose a reason for hiding this comment

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

p3) 전에 논의 많이 하기로는 -1 은 지양하기로 했던 거 같은데, 이걸 사용해야만 이유가 있었을까요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

아! 해당 pr은 논의 이전에 올린 pr입니다..!
논의 이후에도 온보딩 플로우 기디논의가 있어서 플로우 확정이 나면 변경하려고 모든 컴포넌트에서 이동 버튼 구현 안해두었습니다!
현재는 기디 논의가 확정 난 상황이고, 조립시 한 번에 구현 하려고 합니다!! 앞으로의 pr에서 해당 내용 작성해 두겠습니다!

Comment on lines +4 to +10
const errors: string[] = [];

// 영어 또는 숫자 하나라도 포함되면 오류
const alphanumericPattern = /[a-zA-Z0-9]/;
if (alphanumericPattern.test(petName)) {
errors.push(ERROR_MSG.petName.enNum); // 영어 또는 숫자가 포함된 경우
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

p5) 아주 좋은 생각! 상태 많이 둬봤자 신경쓸 것만 많아져요~ 유틸함수 지향 잘 배웠네요! 좋습니다

Copy link
Collaborator

@Leeyoonji23 Leeyoonji23 left a comment

Choose a reason for hiding this comment

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

중간중간 주석처리 덕분에 어떤 식으로 짜여졌는지 흐름 알기 쉬웠습니다 :)
고생하셨습니다 !!!~ 예림업!!!!

Comment on lines +21 to +23
// 유효성 검사 결과
const validationMessages = petName ? validatePetName(petName) : [];
const isValid = petName && validationMessages.length === 0;
Copy link
Collaborator

Choose a reason for hiding this comment

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

p5) 유효성 검사 좋아요 🫠❤️

@yarimu yarimu merged commit d511d65 into develop Jan 17, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 design 사용자 UI 디자인 변경 - ex) CSS ✨ feat 기능 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ Feat ] 반려동물 이름 등록 뷰 구현
4 participants