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

feat(dunning): Fix webhooks events processing and payment request email triggering on PSPs payment attempts #2546

Merged
merged 11 commits into from
Sep 12, 2024

Conversation

ancorcruz
Copy link
Contributor

@ancorcruz ancorcruz commented Sep 6, 2024

Roadmap Task

👉 https://getlago.canny.io/feature-requests/p/send-reminders-for-overdue-invoices

Context

We want to be able to manually request payment of the overdue balance and send emails for reminders.

Description

The goal of this change is trigger PaymentRequest requested email when payable payment status is failed on PSPs payment attempts. The change includes all PSPs (stripe, adyen and gocardless) and payment creation and status update event scenarios.

Also, fixes a deprecation warning on payment request aliased attribute total_amount_cents and CTA translation on Payment Request requested email for German language.

@ancorcruz ancorcruz self-assigned this Sep 6, 2024
@ancorcruz ancorcruz marked this pull request as ready for review September 9, 2024 10:43
@ancorcruz ancorcruz force-pushed the feat/dunning-qa-returns-email-trigger branch from 6fea8e8 to b818f79 Compare September 10, 2024 12:11
@ancorcruz ancorcruz changed the title Feat/dunning qa returns email trigger feat(dunning): Fix webhooks events processing and payment request email triggering on PSPs payment attempts Sep 11, 2024
@rsempe rsempe force-pushed the feat/dunning-qa-returns-email-trigger branch from 874b223 to e966ba8 Compare September 12, 2024 08:54
on Adyen non successful response for the first payment intent,
then the payment request requested email is sent to customer
@ancorcruz ancorcruz force-pushed the feat/dunning-qa-returns-email-trigger branch from f90e423 to f3e1bdd Compare September 12, 2024 13:44
@ancorcruz ancorcruz merged commit 7a56c28 into main Sep 12, 2024
6 checks passed
@ancorcruz ancorcruz deleted the feat/dunning-qa-returns-email-trigger branch September 12, 2024 13:57
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