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.

Sign up for GitHub

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

Jump to bottom

refactor: 로그인 페이지 및 api 리팩토링 #92

Merged
merged 2 commits into from
Sep 24, 2023
Merged

refactor: 로그인 페이지 및 api 리팩토링 #92

merged 2 commits into from
Sep 24, 2023

Conversation

truewayy
Copy link
Member

@truewayy truewayy commented Sep 24, 2023 •
edited
Loading

  1. navigate 사용부 함수화
  2. 기존 useRef로 값 전달하던 것들 useForm 사용하여 값 전달
  3. axios error타입 단언하여(스웨거 api 에러 타입 참조) 에러 시 메시지 호출
  4. form submitting 시에 로그인 버튼 disabled 처리 (button opacity 조절)

점진적으로 개선해볼 예정입니다~
image
image

@truewayy truewayy added this to the 점진적으로 개선하기 milestone Sep 24, 2023
@truewayy truewayy self-assigned this Sep 24, 2023
@netlify
Copy link

netlify bot commented Sep 24, 2023 •
edited
Loading

✅ Deploy Preview for suwiki-test ready!

Name Link
🔨 Latest commit 904bfce
🔍 Latest deploy log https://app.netlify.com/sites/suwiki-test/deploys/65103db31ae34600089b824b
😎 Deploy Preview https://deploy-preview-92--suwiki-test.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.

@tooooo1 tooooo1 merged commit fd2a95f into uswLectureEvaluation:dev Sep 24, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tooooo1 tooooo1 tooooo1 approved these changes

Assignees

@truewayy truewayy

Labels
refactor
Projects
None yet
Milestone
점진적으로 개선하기
Development

Successfully merging this pull request may close these issues.
2 participants
@truewayy @tooooo1