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-be: 이메일 인증 시 이미 가입된 이메일은 예외를 던지도록 변경 #906

Merged
merged 7 commits into from
Oct 23, 2024

Conversation

github-actions[bot]
Copy link
Contributor

Original issue description

목적

이미 가입된 이메일은 이메일 인증을 못하도록 수정

작업 세부사항

  • 예외 추가

참고 사항

아래의 별표줄 밑에 요구사항 ID만 작성해주세요. Prefix 금지!


EMAIL_VALID_03

closes #905

@github-actions github-actions bot added backend 백엔드 fix 버그 수정 labels Oct 23, 2024
@HyungHoKim00 HyungHoKim00 marked this pull request as ready for review October 23, 2024 06:03
Copy link
Contributor Author

1729663446.289289

Copy link
Member

@cutehumanS2 cutehumanS2 left a comment

Choose a reason for hiding this comment

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

EmailFacade 테스트는 추가 안 해도 되나요??


import com.cruru.advice.badrequest.BadRequestException;

public class EmailAlreadySignedUpException extends BadRequestException {
Copy link
Contributor

Choose a reason for hiding this comment

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

EmailConflictException 으로 바꾸고 ConflictException 상속받는건 어떤가유

@Dobby-Kim Dobby-Kim merged commit b8ebff3 into be/develop Oct 23, 2024
5 checks passed
@Dobby-Kim Dobby-Kim deleted the 905-be-EMAIL_VALID_03 branch October 23, 2024 06:31
@Dobby-Kim Dobby-Kim mentioned this pull request Oct 23, 2024
Dobby-Kim added a commit that referenced this pull request Oct 23, 2024
fix-be: applicantCard 조회 시 평가 점수와 갯수가 모든 프로세스에 대해 나오는 버그 수정 (#902)
fix-be: 이메일 인증 시 이미 가입된 이메일은 예외를 던지도록 변경 (#906)
feat-be: RTR 방식 Redis로 변경 (#889)
feat-be: 이메일 전송 템플릿 적용 (#897)
feat-be: 이메일 인증 후, 회원가입 이메일의 인증 여부 확인 (#882)
refactor-be: 식별자에 Tsid 적용 (#856)
feat-be: 이메일 전송 로직 중 지원자 일괄 조회 로직 구현 (#869)
fix-be: github action의 Redis 환경 변수 추가 (#871)
feat-be: 이메일 인증 기능 구현 (#848)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend 백엔드 fix 버그 수정
Projects
Status: 완료
Development

Successfully merging this pull request may close these issues.

3 participants