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

[COZY-232] feat : mail template 생성 #188

Merged
merged 1 commit into from
Nov 26, 2024
Merged

[COZY-232] feat : mail template 생성 #188

merged 1 commit into from
Nov 26, 2024

Conversation

genius00hwan
Copy link
Contributor

⚒️develop의 최신 커밋을 pull 받았나요?

#️⃣ 작업 내용

메일을 이쁘게 바꿨습니다.

동작 확인

image

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
고민사항도 적어주세요.

@github-actions GitHub Actions
Copy link

리뷰해드려요~

MailService.java

  • 개선 1: 새로운 예외 처리 추가

  • 개선 2: 메일 본문 생성 함수 추가

  • 개선 3: 메일 전송 예외 처리 추가

  • 개선 1: 새로운 예외 처리 추가
    새로운 예외 _MAIL_SEND_FAIL와 _CANNOT_FIND_MAIL_FORM가 추가되었습니다. 이들은 메일 전송 중 오류가 발생하거나 메일 템플릿을 찾지 못하는 경우에 사용됩니다.

  • 개선 2: 메일 본문 생성 함수 추가
    makeMailBody 함수가 추가되었습니다. 이 함수는 메일 본문을 생성하고 대학 이름과 인증 코드를 본문에 삽입합니다.

  • 개선 3: 메일 전송 예외 처리 추가
    MessagingException 예외가 처리되도록 추가되었습니다. 이 예외는 메일 전송 중 오류가 발생하는 경우에 발생합니다.

ErrorStatus.java

  • 개선 1: 새로운 예외 추가

  • 개선 2: 새로운 예외 추가

  • 개선 1: 새로운 예외 추가
    _MAIL_SEND_FAIL와 _CANNOT_FIND_MAIL_FORM가 추가되었습니다. 이들은 메일 전송 중 오류가 발생하거나 메일 템플릿을 찾지 못하는 경우에 사용됩니다.

  • 개선 2: 새로운 예외 추가
    _MAIL_SEND_FAIL와 _CANNOT_FIND_MAIL_FORM가 추가되었습니다. 이들은 메일 전송 중 오류가 발생하거나 메일 템플릿을 찾지 못하는 경우에 사용됩니다.

genius00hwan
genius00hwan commented Nov 25, 2024
View reviewed changes
src/main/java/com/cozymate/cozymate_server/domain/mail/service/MailService.java
String emailBody = makeMailBody(authenticationCode,universityName);

try {
MimeMessage message = mailSender.createMimeMessage();
Copy link
Contributor Author

Choose a reason for hiding this comment

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

html사용을 위해 mimeMessage를 사용했습니다.

src/main/java/com/cozymate/cozymate_server/domain/mail/service/MailService.java
@@ -124,4 +137,23 @@ private void validateMailAddress(String mailAddress, String mailPattern) {
}
}

private String makeMailBody(String authenticationCode, String universityName) {
try {
InputStream inputStream = getClass().getClassLoader().getResourceAsStream(
Copy link
Contributor Author

Choose a reason for hiding this comment

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

resources에 mail_form.html을 두고 값들을 렌더링해서 메일 바디를 만듭니다

src/main/resources/mail/mail_form.html
Copy link
Contributor Author

Choose a reason for hiding this comment

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

gpt와 합작하여 만들었습니다. 컬러코드는 델로가 주더라요

Copy link
Contributor

Choose a reason for hiding this comment

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

재능있네요. 합격드리겠습니다.

@genius00hwan genius00hwan self-assigned this Nov 25, 2024
@genius00hwan genius00hwan added the enhancement New feature or request label Nov 25, 2024
jpark0506
jpark0506 approved these changes Nov 25, 2024
View reviewed changes
Copy link
Contributor

@jpark0506 jpark0506 left a comment

Choose a reason for hiding this comment

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

LGTM~ 프런트 코드 희귀하네여

src/main/resources/mail/mail_form.html
Copy link
Contributor

Choose a reason for hiding this comment

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

재능있네요. 합격드리겠습니다.

veronees
veronees approved these changes Nov 26, 2024
View reviewed changes
Copy link
Member

@veronees veronees left a comment

Choose a reason for hiding this comment

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

LGTM~ 굳

eple0329
eple0329 approved these changes Nov 26, 2024
View reviewed changes
Copy link
Member

@eple0329 eple0329 left a comment

Choose a reason for hiding this comment

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

굿

@genius00hwan genius00hwan merged commit dd10b03 into develop Nov 26, 2024
1 check passed
@eple0329 eple0329 deleted the feature/COZY-232 branch November 27, 2024 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@veronees veronees veronees approved these changes

@eple0329 eple0329 eple0329 approved these changes

@jpark0506 jpark0506 jpark0506 approved these changes

@suuu0719 suuu0719 Awaiting requested review from suuu0719 suuu0719 is a code owner

Assignees

@genius00hwan genius00hwan

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@genius00hwan @jpark0506 @eple0329 @veronees