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

이메일 전송 로직 개선 #368

Closed
3 tasks done
SongJaeHoonn opened this issue Jun 8, 2024 · 0 comments · Fixed by #369
Closed
3 tasks done

이메일 전송 로직 개선 #368

SongJaeHoonn opened this issue Jun 8, 2024 · 0 comments · Fixed by #369
Assignees
Labels
🔨 Refactor 코드 수정 및 개선

Comments

@SongJaeHoonn
Copy link
Contributor

SongJaeHoonn commented Jun 8, 2024

Describe

현재 이메일 전송 로직에서는 객체로 만들어야 할 클래스를 inner class로 두기도 하고, @async를 사용하는 과정에서 자가 호출이 일어나는 문제가 있으며, 이메일 전송 큐를 사용하여 스케줄링을 통해 지속적으로 mail을 프로세싱하는 메소드를 호출하고 있습니다.
이에 따라 이메일 전송 로직 개선을 진행합니다.

Tasks

  • EmailTask 위치 변경
  • Async 어노테이션이 적용된 메소드를 컴포넌트로 등록
  • 스케줄링 삭제

ETC

No response

@SongJaeHoonn SongJaeHoonn added the 🔨 Refactor 코드 수정 및 개선 label Jun 8, 2024
@SongJaeHoonn SongJaeHoonn self-assigned this Jun 8, 2024
limehee pushed a commit that referenced this issue Jun 9, 2024
* refactor: EmailTask 클래스 domain 패키지로 위치 이동 #368

* refactor: Async 관련 메소드 다른 컴포넌트로 분리 #368
@limehee limehee linked a pull request Jun 9, 2024 that will close this issue
@limehee limehee closed this as completed Jun 9, 2024
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 a pull request may close this issue.

2 participants