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

이메일 전송 로직 개선 완료 #369

Merged
merged 2 commits into from
Jun 9, 2024
Merged

이메일 전송 로직 개선 완료 #369

merged 2 commits into from
Jun 9, 2024

Conversation

SongJaeHoonn
Copy link
Contributor

@SongJaeHoonn SongJaeHoonn commented Jun 8, 2024

Summary

#368

코드를 알아보기 쉽게 개선하고, 이메일 전송 과정에서 스케줄링을 제거하고 비동기 메소드 사용 방식을 수정했습니다.

Tasks

  • EmailTask 분리
  • 스케줄링 제거
  • Async 관련 메소드 사용 방식 수정

ETC

Screenshot

email1-2 전체메일전송

파일 없이 메일 전송 결과
일반메일전송 파일 없이 실제
파일과 함께 메일 전송 결과
일반메일전송 파일 있이 실제
일반 메일 전송, 전체 메일 전송 둘 다 결과가 같게 나왔습니다.

@SongJaeHoonn SongJaeHoonn added the 🔨 Refactor 코드 수정 및 개선 label Jun 8, 2024
@SongJaeHoonn SongJaeHoonn self-assigned this Jun 8, 2024
@SongJaeHoonn SongJaeHoonn requested a review from limehee as a code owner June 8, 2024 11:33
@mingmingmon
Copy link
Collaborator

블로그를 읽고 코드를 확인하니 많은 고민을 하고 리팩토링을 진행하신 것 같습니다! 코드도 깔끔하고
코어팀의 현재 이메일 전송 정책에 알맞게 수정 한 것 같습니다!

다만 스웨거 테스트 시에 이메일 전송을 하고 실제로 이메일을 잘 받았는지, 템플릿은 잘 적용이 되는지, 첨부파일 전송도 잘 되는지를 확인할 수 있는 화면을 추가를 요청드립니다!

@limehee limehee requested a review from mingmingmon June 8, 2024 16:25
@limehee
Copy link
Collaborator

limehee commented Jun 8, 2024

처음 받은 도전과제였음에도 불구하고 작업을 훌륭하게 마무리하신 것 같습니다. 고생 많으셨어요.
위에 코멘트대로 스웨거 테스트에 대한 결과도 첨부해주시면 확인 후에 develop 브랜치에 포함하겠습니다.

@limehee limehee merged commit 2bdcd64 into develop Jun 9, 2024
1 check passed
@limehee limehee deleted the refactor/#368 branch June 9, 2024 11:14
@limehee limehee linked an issue Jun 9, 2024 that may be closed by this pull request
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 Refactor 코드 수정 및 개선
Projects
None yet
Development

Successfully merging this pull request may close these issues.

이메일 전송 로직 개선
3 participants