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

update:: 비밀번호 재설정 훅 분리, apis 작성 #140 #143

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

greenfrog616
Copy link
Member

USER비밀번호 재설정

훅을 분리하여 이메일 인증과 비밀번호 변경 함수를 불러오도록 작성하고 apis를 작성해서 서버로 post를 보내서 이메일 인증,비밀 번호 변경한 내용을 post로 보내도록 작성했습니다.

훅을 분리하여 이메일 인증과 비밀번호 변경 함수를 불러오도록 작성하고 apis를 작성해서 서버로 post를 보내서 이메일 인증,비밀 번호 변경한 내용을 post로 보내도록 작성했습니다.
@greenfrog616 greenfrog616 added the CHORE 수정 사항 label Jun 18, 2024
@greenfrog616 greenfrog616 self-assigned this Jun 18, 2024
@greenfrog616 greenfrog616 reopened this Jun 19, 2024
@greenfrog616 greenfrog616 requested review from Bae0203 and sueeeeh and removed request for Bae0203 and sueeeeh June 20, 2024 05:05
Copy link
Member

@Bae0203 Bae0203 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다.
추가로 풀리퀘를 올리실 때 수정된 사항만 올리는 것이 아닌
이슈번호(ex. #99)

대제목(ex. 원서접수 5번 모바일 퍼블리싱)
Preview (프리뷰는 없어도 됩니다)
(이미지들)

수정 및 추가된 사항들
내용+기능설명 (ex. useWriteNav 추가(브라우저 width 반환))
위 내용을 다 올려주셔야 원활한 코드리뷰가 진행되니 다음에 올리실때는 위 내용을 전부 추가해주시길 바랍니다.
감사합니다.

// const [isEmailAuth, SetIsEmailAuth] = useState(false)
const isEmailAuth = false
const [isEmailAuth, setIsEmailAuth] = useState(false)
const repository = ChangePasswordRepositoryImpl
Copy link
Member

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
CHORE 수정 사항
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants