Skip to content

email 도메인 리펙토링, partner(제휴업체) 기능 추가 #219

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
Merged
X1n9fU merged 40 commits into from
Jul 2, 2025
Merged

email 도메인 리펙토링, partner(제휴업체) 기능 추가 #219

X1n9fU merged 40 commits into from
Jul 2, 2025

Conversation

@X1n9fU
Copy link
Contributor

@X1n9fU X1n9fU commented Jul 2, 2025

#️⃣ 연관된 이슈

ex) #이슈번호, #이슈번호
#218 #211 #208

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

스크린샷 (선택)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

doma17 and others added 30 commits June 23, 2025 17:04
@doma17
refactor: Email 도메인 리팩토링
7612c6f 
- EmailAuthService와 EmailSendService 계층 분리
- 과거 EmailAuthService을 JoinEmailAuthService와 PasswordResetEmailService로 책임 분리
- EmailSendService에서 EmailTemplateService로 변경을 통해 이메일 전송 역할 명확화
@doma17
remove: EmailAuthService, EmailSendService 제거
6c9e6a4
@doma17
test: Email Service 테스트 추가
9bfa775 
아래와 같은 세가지에 대해서 테스트를 작성했습니다.
- EmailTemplateServiceTest
- JoinEmailAuthServiceTest
- PasswordResetEmailServiceTest

테스트시 객체 생성 메서드가 없기에 두개의 DTO 클래스에 Builder를 추가했습니다.
- JoinEmailCheckRequestDto
- JoinEmailSendRequestDto
@doma17
refactor: #207 수정사항 적용
fb30b48 
- EmailController : 불필요한 공백 제거
- EmailTemplateService : 예외처리 추가
- EmailTemplateServiceTest : 가독성 개선
- JoinEmailAuthService : 잘못된 주석 제거와 가독성 향상
- PasswordResetEmailService : 잘못된 주석 제거
@doma17
refactor: #207 수정사항 적용 2
6d00629 
- EmailTemplateFailException 으로 예외처리 추가
@doma17
fixup! refactor: #207 수정사항 적용 2
9de88f6
@doma17
refactor: #207 수정사항 적용 3
5915b3f 
비밀번호 재인증 로직 중 기존 인증정보가 존재하지 않을 때에는 예외처리
- EmailPasswordResetFailException 으로 예외처리 추가
@doma17
test: #207 수정사항 적용 5
193393e 
수정된 기능 상황에 맞게 테스트 수정
@doma17
Merge pull request #207 from CodIN-INU/refactor/email-feat
8146363 
refactor : Email 도메인 리팩토링
@doma17
test: #207 수정사항 적용 6
04b01cb 
- EmailTemplateFailException : import 최적화
- EmailTemplateFailException : 테스트 예외처리 수정
@doma17
Merge pull request #208 from CodIN-INU/refactor/email-feat
3ff251a 
refactor : Email 도메인 리팩토링 / 테스트 예외처리 수정 추가
@X1n9fU
fix : GlobalException, ErrorCode interface 설정. ChatException 제작
c76b9a5
@X1n9fU
fix : 불필요한 Getter 삭제 및 GlobalExceptionHandler에 ChattingException 추가
6ba843a
@X1n9fU
style : 코드 스타일 일치를 위한 공백 제거
8fbb531
@X1n9fU
Merge pull request #211 from CodIN-INU/feat/exception
ac09648 
fix : GlobalException, ErrorCode interface 설정. ChatException 제작
@doma17 @X1n9fU
refactor: Email 도메인 리팩토링
d9e33b8 
- EmailAuthService와 EmailSendService 계층 분리
- 과거 EmailAuthService을 JoinEmailAuthService와 PasswordResetEmailService로 책임 분리
- EmailSendService에서 EmailTemplateService로 변경을 통해 이메일 전송 역할 명확화
@doma17 @X1n9fU
remove: EmailAuthService, EmailSendService 제거
7c9007e
@doma17 @X1n9fU
test: Email Service 테스트 추가
a252670 
아래와 같은 세가지에 대해서 테스트를 작성했습니다.
- EmailTemplateServiceTest
- JoinEmailAuthServiceTest
- PasswordResetEmailServiceTest

테스트시 객체 생성 메서드가 없기에 두개의 DTO 클래스에 Builder를 추가했습니다.
- JoinEmailCheckRequestDto
- JoinEmailSendRequestDto
@doma17 @X1n9fU
refactor: #207 수정사항 적용
878ffa1 
- EmailController : 불필요한 공백 제거
- EmailTemplateService : 예외처리 추가
- EmailTemplateServiceTest : 가독성 개선
- JoinEmailAuthService : 잘못된 주석 제거와 가독성 향상
- PasswordResetEmailService : 잘못된 주석 제거
@doma17 @X1n9fU
refactor: #207 수정사항 적용 2
6ad71a1 
- EmailTemplateFailException 으로 예외처리 추가
@doma17 @X1n9fU
fixup! refactor: #207 수정사항 적용 2
e2db74e
@doma17 @X1n9fU
refactor: #207 수정사항 적용 3
63ad8ec 
비밀번호 재인증 로직 중 기존 인증정보가 존재하지 않을 때에는 예외처리
- EmailPasswordResetFailException 으로 예외처리 추가
@doma17 @X1n9fU
test: #207 수정사항 적용 5
03b6114 
수정된 기능 상황에 맞게 테스트 수정
@doma17 @X1n9fU
test: #207 수정사항 적용 6
5a0de40 
- EmailTemplateFailException : import 최적화
- EmailTemplateFailException : 테스트 예외처리 수정
@X1n9fU
refactor : Info package 내부 구조 평탄화
7cb6206
@X1n9fU
feat : Partner(제휴 업체) Entity 설계 및 Read 기능 구현
a78eaa4
@X1n9fU
fix : GlobalException, ErrorCode interface 설정. ChatException 제작
1d54470
@X1n9fU
fix : 불필요한 Getter 삭제 및 GlobalExceptionHandler에 ChattingException 추가
456f8bd
@X1n9fU
style : 코드 스타일 일치를 위한 공백 제거
e4635d0
@X1n9fU
refactor : InfoException, InfoErrorCode 생성 및 GlobalExceptionHandler 에 추가
4e665a9
X1n9fU added 10 commits July 1, 2025 02:30
@X1n9fU
feat : Partner create 로직 제작
987cd1d
@X1n9fU
fix : Scheme 작성 및 Department의 JSON 매핑 로직 추가
90795f3
@X1n9fU
feat : 제휴업체 삭제 로직 추가 및 생성 시 이미지 업로드 진행
9ba2370
@X1n9fU
test : 제휴업체 기능 반환, 생성, 삭제 로직 테스트 코드 작성
f1f851c
@X1n9fU
fix : JsonCreator name과 value 모두 조회하도록 수정, JsonValue에서 description이 아…
a55c3c1 
…닌 value로 수정
@X1n9fU
feat : 제휴업체 위치 location 필드 추가
84e61e0
@X1n9fU
fix : 이미지가 없어도 생성 가능하도록 로직 수정
e6c1c49
@X1n9fU
refactor : 코드 인라인화, 불필요한 트랜잭션 삭제
11d2eb9
@X1n9fU
refactor : 잠시 주석처리, chat 기능 추합 후 주석 해제
87db5f8
@X1n9fU
Merge pull request #218 from CodIN-INU/215-feat-제휴업체-기능-개발
e7700ce 
215 feat 제휴업체 기능 개발
@X1n9fU X1n9fU self-assigned this Jul 2, 2025
@X1n9fU X1n9fU added the 🌏 Deploy 배포 관련 label Jul 2, 2025
@X1n9fU X1n9fU merged commit 3848628 into main Jul 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@X1n9fU X1n9fU

Labels

🌏 Deploy 배포 관련

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@X1n9fU @doma17