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

Send 'renewal received pending payment' emails from Notify #989

Merged
merged 2 commits into from
May 6, 2021

Conversation

tobyprivett
Copy link
Contributor

@tobyprivett tobyprivett commented May 5, 2021

https://eaflood.atlassian.net/browse/RUBY-1390

This (should) remove the last of the RenewalMailer files

@tobyprivett tobyprivett added the enhancement New feature or request label May 5, 2021
@tobyprivett tobyprivett force-pushed the feature/send-renewal-pending-payment-with-notify branch 2 times, most recently from 3f9d8ff to f641c93 Compare May 5, 2021 15:15
@tobyprivett tobyprivett force-pushed the feature/send-renewal-pending-checks-email-with-notify branch from b658858 to cdff7fa Compare May 6, 2021 09:22
Base automatically changed from feature/send-renewal-pending-checks-email-with-notify to main May 6, 2021 12:30
@tobyprivett tobyprivett force-pushed the feature/send-renewal-pending-payment-with-notify branch from f641c93 to d0c2e2d Compare May 6, 2021 12:32
# An older incarnation of this spec checked that ActionMailer
# did not send *any* emails.
# Here we check to see if the notification service has been invoked,
# which should tell us if *any* emails have been sent.
Copy link
Member

Choose a reason for hiding this comment

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

👏

@tobyprivett tobyprivett merged commit 5e1031b into main May 6, 2021
@tobyprivett tobyprivett deleted the feature/send-renewal-pending-payment-with-notify branch May 6, 2021 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants