-
Notifications
You must be signed in to change notification settings - Fork 40
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
18924 - Fix payment_date for Disbursements, EJV, PAD, ONLINE BANKING. Fix di… #1344
Conversation
…bursement_date. Fix refund_date for EJV, Disbursements.
@@ -155,9 +155,7 @@ def update_invoice_to_refunded_or_paid(cls, invoice: InvoiceModel): | |||
refund.gl_posted = datetime.now() | |||
refund.save() | |||
invoice.invoice_status_code = InvoiceStatus.REFUNDED.value | |||
invoice.refund_date = datetime.now() |
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.
Don't want to lose this data, as we're just shifting over GL's here.
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #1344 +/- ##
==========================================
- Coverage 91.45% 87.22% -4.24%
==========================================
Files 186 40 -146
Lines 11319 3005 -8314
==========================================
- Hits 10352 2621 -7731
+ Misses 967 384 -583
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
…sbursement_date. Fix refund_date for EJV, Disbursements.
Issue #:
bcgov/entity#18924
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).