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

[ Fix ] 후배 학교 메일 인증 API 변경사항 반영 #332

Merged
merged 2 commits into from
Nov 4, 2024

Conversation

lydiacho
Copy link
Collaborator

#️⃣ Related Issue

Closes #331

✅ Done Task

  • API 변경 사항 반영
  • 작동 테스트

💎 PR Point

기존에 사용하던 Open API가 한번 사용한 메일주소는 재사용하지 못하는 이슈로 테스트환경에 매우 부적합했는데요,
그래서 서버 측에서 학교명 - 학교메일주소도메인 데이터를 DB에 직접 넣어서 검증하는 방식으로 API가 수정되었습니다
그 과정에서 API 살짝 바뀌어서 해당 변경사항 반영해주었고,
작업 후 학교 메일 인증 정상적으로 이루어지는지도 테스트 완료했습니다! (아래 영상 참고)

이제 후배 온보딩 끝까지 갈 수 있다 ! 야호 ! 🎉

📸 Screenshot

i.e.e.e.2024-10-31.i.i.1.02.37.mov

@lydiacho lydiacho added ✨ Feat 새로운 기능 추가 (새로운 구현) labels Oct 31, 2024
@lydiacho lydiacho requested review from ijieun and se0jinYoon October 31, 2024 04:07
@lydiacho lydiacho self-assigned this Oct 31, 2024
Copy link
Collaborator

@se0jinYoon se0jinYoon left a comment

Choose a reason for hiding this comment

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

lgtm
유효하지 않은 이메일이라고 뜬 후 다시 입력해서 유효한 메일일 경우 빨간 보더값 없애주도록 해야할 것 같슴다!
이번 피알 해당 코드가 아니여서 리뷰를 여기다 다네요 , ,

바뀐 로직 확인했습니다 수고하셨어요!

@lydiacho
Copy link
Collaborator Author

@se0jinYoon 좋아요!
메일 유효 여부를 알 수 있는 시점이 '인증번호 전송' 이때여서 현재 구현된 모습이 유효한 메일일 경우 보더를 없애주는 형태예요!
근데 그때까지 빨간 박스 남아있는게 넘 어색한 것 같아서 닉네임 입력 때처럼 입력 수정하면 에러박스 사라지도록 했습니당
작업!

2024-10-31.10.55.00.mov

Copy link
Collaborator

@ijieun ijieun left a comment

Choose a reason for hiding this comment

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

LGTM

@lydiacho lydiacho merged commit fed576d into develop Nov 4, 2024
1 check passed
@lydiacho lydiacho deleted the feat/#331/univEmail branch November 4, 2024 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feat 새로운 기능 추가 (새로운 구현) size/m
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ Feat ] 학교메일인증 API 변경사항 반영
3 participants