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

15674 - Enhancements to the daily PAD invoice email #3310

Merged
merged 6 commits into from
Mar 20, 2025
Merged

Conversation

Jxio
Copy link
Collaborator

@Jxio Jxio commented Mar 19, 2025

Issue #:
bcgov/entity#15674

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the sbc-auth license (Apache 2.0).

@Jxio Jxio self-assigned this Mar 19, 2025
@@ -1,4 +1,7 @@
# Your {{ account_number }}: {{ account_name_with_branch }} account completed ${{ invoice_total }} in transactions during business hours on {{ invoice_process_date }}.
# This email confirms a recent transaction on you account {{ account_number }}: {{ account_name_with_branch }}.
Copy link
Collaborator

@seeker25 seeker25 Mar 20, 2025

Choose a reason for hiding this comment

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

This email confirms recent transaction(s) on your* account

Check with Ethan -^

@@ -8,6 +11,10 @@

Your account will be suspended and charged a ${{ nsf_fee }} dishonoured bank instrument fee for every failed payment.

Log in to BC Registries and Online Services.
[Log in to view transaction detail]({{ transaction_url }})
Copy link
Collaborator

@seeker25 seeker25 Mar 20, 2025

Choose a reason for hiding this comment

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

transaction details*?

Check with Ethan -^

Copy link
Collaborator

@seeker25 seeker25 left a comment

Choose a reason for hiding this comment

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

LGTM

@seeker25
Copy link
Collaborator

seeker25 commented Mar 20, 2025

Account Mailer now has a dependency on bcgov/sbc-pay#1929

Small lint issue as well

tests/unit/test_worker_queue.py:247:16: F541 f-string is missing placeholders

Copy link
Collaborator

@seeker25 seeker25 left a comment

Choose a reason for hiding this comment

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

LGTM nice work

@Jxio Jxio marked this pull request as ready for review March 20, 2025 15:53
@Jxio Jxio merged commit a4be6c3 into bcgov:main Mar 20, 2025
6 of 8 checks passed
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