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

Refactor/signin and signup #144

Merged
merged 4 commits into from Aug 9, 2021
Merged

Refactor/signin and signup #144

merged 4 commits into from Aug 9, 2021

Conversation

simian114
Copy link
Collaborator

signup과 signin 리팩토링 진행했습니다.

signin은 useCallback만 지워서 신경쓰지 않으셔도 될거같습니다.

signup 코드 중복 제거, validation 과정 등... 많은 부분 리팩토링했습니다.

아직 카드는 진행하지 않았습니다.
카드에 있는 로직 전부를 수정해야하기 때문에 일단 카드쪽의
validation 은 주석했습니다.
@simian114 simian114 self-assigned this Aug 8, 2021
@simian114 simian114 added this to In progress in JARANDA via automation Aug 8, 2021
@netlify
Copy link

netlify bot commented Aug 8, 2021

✔️ Deploy Preview for this-is-jaranda-develop ready!

🔨 Explore the source changes: 1e6c436

🔍 Inspect the deploy log: https://app.netlify.com/sites/this-is-jaranda-develop/deploys/610ffe6a1786b80007d8339a

😎 Browse the preview: https://deploy-preview-144--this-is-jaranda-develop.netlify.app/

@dltkr12311 dltkr12311 merged commit ed40a75 into develop Aug 9, 2021
JARANDA automation moved this from In progress to Done Aug 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants