-
Notifications
You must be signed in to change notification settings - Fork 1
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
Conversation
오 작업 수고하셨습니다! 다음 주 회의에서 리뷰 후 머지하면 될 것 같아요. + 로그가 모두 영어로 바뀌었군요!! 작업해 주셔서 감사해요! |
네넵, 그 문제가 맞는 것 같습니다. 근데 dev 서버에서 NODE_ENV 값이 production이 아니라 development로 되어 있어서 테스트가 잘 될지 모르겠네요. (채널톡 서버 IP 주소가 등록되어 있는지도 휠장님께 확인이 필요할 것 같습니다.) 동방에서 SPARCS_AP_2로 연결하시면 로컬에서도 테스트 해보실 수 있습니다! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 메일 Layout 관련해서 코멘트 하나 남겼는데, 머지 전 확인 부탁드립니다..!
자꾸 코멘트 남겨서 죄송합니다 ㅠㅠ 메일이 잘 전송됐는지 확인하기 위해, cc에 [email protected] 를 넣고 메일을 전송하면 어떨까요? 이렇게 하면 Taxi팀 모두가 해당 메일 내용을 확인할 수 있습니다. |
@kmc7468 아닙니다~! 코멘트는 항상 환영합니다 :) |
…485-no-show-report-email
cc까지 추가 완료했습니다~! 채널톡 관련 front 작업 및 prod 환경에서 작동 여부 추후 확인해보면 좋을 것 같습니다 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!! 감사합니다!
Summary
It closes #485
(문구같은 경우는 리뷰를 통해 검토가 필요할 것 같습니다.)
Images or Screenshots
Further Work