Skip to content

수환 3차QA#194

Open
ksooh95 wants to merge 11 commits intodevfrom
Fix/soohwanQA3
Open

수환 3차QA#194
ksooh95 wants to merge 11 commits intodevfrom
Fix/soohwanQA3

Conversation

@ksooh95
Copy link
Contributor

@ksooh95 ksooh95 commented Dec 17, 2024

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

✨ 과제 내용

📸 스크린샷(선택)

📌 관련 이슈

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들

@netlify
Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for zingy-strudel-7dad97 ready!

Name Link
🔨 Latest commit d44d8bc
🔍 Latest deploy log https://app.netlify.com/sites/zingy-strudel-7dad97/deploys/67763d3701658c000864f007
😎 Deploy Preview https://deploy-preview-194--zingy-strudel-7dad97.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

const [showConfirmPassword, setShowConfirmPassword] =
useState<boolean>(false);
const [isPasswordMatched, setIsPasswordMatched] = useState<boolean>(true);
// 비밀번호 유효성 검사
Copy link
Contributor

Choose a reason for hiding this comment

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

앗 수환님 리액트 훅폼에서 validate 기능이 있는것으로 알고있는데 따로 처리 하신 이유가 있을까요오옹~~~?

data: info,
isLoading,
refetch,
} = useQuery<IUserProfile>({
Copy link
Contributor

Choose a reason for hiding this comment

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

이부분도 혹시 나중에 시간적 여유가 되신다면 api > apis > queryHooks 에 작성하고 import 해서 쓰신다면 훨씬 깔끔해 질것같습니다ㅎㅎ!🤓

Copy link
Contributor

@picapipicca picapipicca left a comment

Choose a reason for hiding this comment

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

저는 모두 확인했습니다~!!! 수고하셨습니다 수환님!

Copy link
Contributor

@Jin-Chanyong Jin-Chanyong left a comment

Choose a reason for hiding this comment

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

저도 확인했습니다! 고생하셨어요!

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.

3 participants