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

DEV-1086: use new notification script; pass SMTP_HOST #128

Merged
merged 1 commit into from
May 7, 2024

Conversation

aelkiss
Copy link
Member

@aelkiss aelkiss commented May 7, 2024

No description provided.

@aelkiss
Copy link
Member Author

aelkiss commented May 7, 2024

FYI @moseshll -- I hit merge too early on the other PR and missed needing to update the script the wrapper was calling. I will still go ahead with merging and deploying this -- it won't run until the beginning of June, so if there's anything we need to adjust we have time.

@aelkiss aelkiss merged commit d89fe70 into main May 7, 2024
1 check passed
@coveralls
Copy link

Coverage Status

coverage: 98.808%. remained the same
when pulling 3a3eed7 on DEV-1086-monthly-reports
into a17f379 on main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants