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

#40 휴대폰번호 인증 #44

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

#40 휴대폰번호 인증 #44

wants to merge 5 commits into from

Conversation

SEONMl
Copy link
Contributor

@SEONMl SEONMl commented Sep 9, 2024 •
edited
Loading

기능

#40

  • 문자 전송 대행 : CoolSMS, 건당 20원, 선불 예수금 10,000원부터 충전해서 사용해야 됨
    - 070 번호 발급 : 아톡, 번호 대여비 월 10,000원 <- 아직 미적용.. 개인번호로 사용中
    • 사업자등록이 필요하다고 하네요 그냥 개인번호 사용하겠습니다
  • 문자 전송 후 redis에 인증코드 저장
  • redis는 테스트 못해봐서 기도메타입니다.........🙏

image

@SEONMl SEONMl added the 기능 구현 기능 구현 label Sep 9, 2024
@SEONMl SEONMl self-assigned this Sep 9, 2024
@SEONMl SEONMl requested a review from csct3434 September 9, 2024 15:36
SEONMl
SEONMl commented Sep 10, 2024
View reviewed changes
src/main/java/com/example/temp/user/service/impl/VerifyCertificationCodeServiceImpl.java Outdated

@Override
public void doService(String code, String phoneNumber) {
if (hasKey(phoneNumber)) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

!hasKey 로 수정필요

@SEONMl
fix: if 조건문 수정
f923d94 
redis에 번호가 없어야 exception
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@csct3434 csct3434 Awaiting requested review from csct3434

Assignees

@SEONMl SEONMl

Labels
기능 구현 기능 구현
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@SEONMl