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

#485 노쇼 대응 이메일 템플릿 추가 및 자동화 #488

Merged
merged 11 commits into from
Mar 21, 2024

Conversation

chlehdwon
Copy link
Contributor

@chlehdwon chlehdwon commented Mar 6, 2024

Summary

It closes #485

  • no-show 이메일 템플릿을 생성하고, 이를 전송합니다.
    (문구같은 경우는 리뷰를 통해 검토가 필요할 것 같습니다.)
  • 추가적으로 기존의 로그 중 한글 로그를 모두 영어로 통합하였습니다.

Images or Screenshots

  • no-settlement 이메일
image
  • no-show 이메일
image

Further Work

  • Do something...

@chlehdwon chlehdwon self-assigned this Mar 6, 2024
@chlehdwon chlehdwon requested review from withSang, kmc7468 and 14KGun and removed request for withSang and kmc7468 March 6, 2024 14:13
@kmc7468
Copy link
Member

kmc7468 commented Mar 7, 2024

오 작업 수고하셨습니다! 다음 주 회의에서 리뷰 후 머지하면 될 것 같아요.

+ 로그가 모두 영어로 바뀌었군요!! 작업해 주셔서 감사해요!

@chlehdwon
Copy link
Contributor Author

해당 이슈는 모두 해결 완료하였습니다!
다만 하나 걸리는 건 gmail에서 asset이 잘 뜨는지 테스트를 해보고 싶은데 로컬에서 mock 대신 실제 node mailer를 사용할 경우 ip 주소 문제로 메일이 발송되지 않는 문제가 있었습니다. dev 서버에 올리고 테스트 진행해야 할 것 같아요!

제가 잘 파악한게 맞을까요? @xMHW @kmc7468
해당 에러 로그 같이 첨부합니다.
image

@kmc7468
Copy link
Member

kmc7468 commented Mar 16, 2024

네넵, 그 문제가 맞는 것 같습니다. 근데 dev 서버에서 NODE_ENV 값이 production이 아니라 development로 되어 있어서 테스트가 잘 될지 모르겠네요. (채널톡 서버 IP 주소가 등록되어 있는지도 휠장님께 확인이 필요할 것 같습니다.)

동방에서 SPARCS_AP_2로 연결하시면 로컬에서도 테스트 해보실 수 있습니다!

Copy link
Member

@kmc7468 kmc7468 left a comment

Choose a reason for hiding this comment

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

LGTM! 메일 Layout 관련해서 코멘트 하나 남겼는데, 머지 전 확인 부탁드립니다..!

@kmc7468
Copy link
Member

kmc7468 commented Mar 17, 2024

자꾸 코멘트 남겨서 죄송합니다 ㅠㅠ 메일이 잘 전송됐는지 확인하기 위해, cc에 [email protected] 를 넣고 메일을 전송하면 어떨까요? 이렇게 하면 Taxi팀 모두가 해당 메일 내용을 확인할 수 있습니다.

@chlehdwon
Copy link
Contributor Author

chlehdwon commented Mar 19, 2024

@kmc7468 아닙니다~! 코멘트는 항상 환영합니다 :)
생각해보니 from[email protected] 인지라 이미 발송할 때 알림이 가지 않나요?? 저는 이번에 메일 보낸거 다 알림 확인 했어서 cc가 필요한지 궁금합니다.

@chlehdwon chlehdwon requested a review from kmc7468 March 19, 2024 12:27
@chlehdwon
Copy link
Contributor Author

chlehdwon commented Mar 19, 2024

cc까지 추가 완료했습니다~! 채널톡 관련 front 작업 및 prod 환경에서 작동 여부 추후 확인해보면 좋을 것 같습니다
+재논의 후 다시 롤백 했습니다.

Copy link
Member

@kmc7468 kmc7468 left a comment

Choose a reason for hiding this comment

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

LGTM!! 감사합니다!

@chlehdwon chlehdwon merged commit 7060fd5 into dev Mar 21, 2024
1 check passed
@chlehdwon chlehdwon deleted the #485-no-show-report-email branch March 21, 2024 11:59
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.

노쇼 대응 이메일 템플릿 추가 및 자동화
2 participants