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

BE/#235 알림 발생 시 이메일 전송 기능 추가 #254

Merged
merged 7 commits into from
Oct 2, 2023
Merged

Conversation

kimhalin
Copy link
Contributor

@kimhalin kimhalin commented Oct 2, 2023

🛠️ 변경사항

  • 메일 전송 기능 구현을 위한 의존성 추가 (build.gradle)
    • thymeleaf, mail
  • 이메일 전송을 처리하는 MailingService 추가
  • 이메일에 사용할 템플릿 html 및 이미지 추가 (resources)
  • Notification 데이터를 저장하는 로직을 수행하는 Notification의 addNotification 메서드에 이메일을 전송하는 메서드 호출 로직 추가
    • 이메일 전송은 오랜 시간이 걸리므로 비동기로 동작하도록 했습니다 (@async 사용)


☝️ 유의사항

이메일 전송 기능은 알림이 발생하는 즉시 이메일을 전송하도록 구현했습니다
따라서 이메일의 isEmailSent 컬럼을 삭제했습니다! (배치를 돌려서 짧은 시간을 간격으로 전송하는 것을 고민했기 떄문에 존재했던 컬럼)



👀 참고자료

https://velog.io/@kimhalin/%EC%9D%B4%EB%A9%94%EC%9D%BC-%EC%A0%84%EC%86%A1%EC%9D%84-%ED%8F%AC%ED%95%A8%ED%95%9C-%EC%95%8C%EB%A6%BC-%EA%B8%B0%EB%8A%A5-%EA%B5%AC%ED%98%84%ED%95%98%EA%B8%B0with-Spring-Boot



❗체크리스트

  • 하나의 메소드는 최소의 기능만 하도록 설정했나요?
  • 수정 가능하도록 유연하게 작성했나요?
  • 필요 없는 import문이나 setter 등을 삭제했나요?
  • 기존의 코드에 영향이 없는 것을 확인하였나요?

@kimhalin kimhalin self-assigned this Oct 2, 2023
@kimhalin kimhalin linked an issue Oct 2, 2023 that may be closed by this pull request
2 tasks
@github-actions
Copy link

github-actions bot commented Oct 2, 2023

Unit Test Results

88 tests  ±0   88 ✔️ ±0   5s ⏱️ ±0s
19 suites ±0     0 💤 ±0 
19 files   ±0     0 ±0 

Results for commit b967b5c. ± Comparison against base commit 126c359.

♻️ This comment has been updated with latest results.

@kimhalin kimhalin closed this Oct 2, 2023
@kimhalin kimhalin reopened this Oct 2, 2023
Copy link
Collaborator

@youKeon youKeon left a comment

Choose a reason for hiding this comment

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

확인했습니다!! TEST_PROPERTIES에 mail 부분 환경변수 값만 추가해주시면 될 것 같습니다!

@kimhalin
Copy link
Contributor Author

kimhalin commented Oct 2, 2023

@youKeon 했습니당

Copy link
Collaborator

@youKeon youKeon left a comment

Choose a reason for hiding this comment

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

LGTM...🥲

@kimhalin kimhalin merged commit 655f715 into feature Oct 2, 2023
4 of 5 checks passed
@kimhalin kimhalin deleted the BE/#235 branch October 2, 2023 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] 알림 발생 시, 이메일 전송 기능 구현
2 participants