-
Notifications
You must be signed in to change notification settings - Fork 39
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
17507 - Receipt Creation when NSF Account Paid and Unlocked #1319
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1319 +/- ##
==========================================
- Coverage 91.45% 91.09% -0.37%
==========================================
Files 186 20 -166
Lines 11319 292 -11027
==========================================
- Hits 10352 266 -10086
+ Misses 967 26 -941
Flags with carried forward coverage won't be shown. Click here to find out more. |
@@ -631,7 +633,13 @@ def _publish_queue_message(self, payload): | |||
f'Notification to Queue failed for the Account Mailer : {payload}.', | |||
level='error') | |||
|
|||
def _create_account_event_payload(self, event_type: str, include_pay_info: bool = False): | |||
def _create_account_event_payload( |
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.
too many params
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
Covered by #1377 |
Issue #:17507
bcgov/entity#17507
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-pay license (Apache 2.0).