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

17382 - Update the Wording based on PAD Receipts #1281

Merged
merged 1 commit into from
Oct 17, 2023
Merged

Conversation

Jxio
Copy link
Collaborator

@Jxio Jxio commented Oct 16, 2023

Issue #:
bcgov/entity#17382

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).

@Jxio Jxio self-assigned this Oct 16, 2023
@Jxio Jxio requested review from seeker25 and ochiu as code owners October 16, 2023 22:52
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@codecov
Copy link

codecov bot commented Oct 16, 2023

Codecov Report

Merging #1281 (6da80be) into main (79924ce) will increase coverage by 0.03%.
Report is 23 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1281      +/-   ##
==========================================
+ Coverage   91.45%   91.48%   +0.03%     
==========================================
  Files         186      177       -9     
  Lines       11319    10478     -841     
==========================================
- Hits        10352     9586     -766     
+ Misses        967      892      -75     
Flag Coverage Δ
paymentreconciliationsqueue ?
reportapi 91.09% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
.../payment-jobs/tasks/statement_notification_task.py 32.78% <ø> (+0.96%) ⬆️
pay-api/src/pay_api/models/__init__.py 100.00% <ø> (ø)
pay-api/src/pay_api/models/invoice.py 100.00% <ø> (ø)
pay-api/src/pay_api/models/payment.py 95.87% <ø> (ø)
pay-api/src/pay_api/models/statement.py 97.61% <ø> (ø)
...api/src/pay_api/resources/v1/account_statements.py 100.00% <ø> (ø)
pay-api/src/pay_api/services/payment.py 97.02% <ø> (ø)
pay-api/src/pay_api/services/statement.py 96.87% <ø> (-0.04%) ⬇️
pay-api/src/pay_api/utils/converter.py 100.00% <ø> (ø)
pay-api/src/pay_api/utils/enums.py 100.00% <ø> (ø)
... and 1 more

... and 9 files with indirect coverage changes

@ochiu
Copy link
Collaborator

ochiu commented Oct 17, 2023

@Jxio I noticed in the ticket UX pin it shows "Payment Method" text and it mentions about changing the label to EFT. Is this required for your PR?

@Jxio
Copy link
Collaborator Author

Jxio commented Oct 17, 2023

bcgov/entity#17382

yes, but the payment method text is from api request payload body, I need to update other repo which send the request I think.
just checked the spot grab paymentMethodDescription: https://github.com/Jxio/sbc-pay/blob/8ad8dae155f341dfc3306d1b387d0f9a5eb2c0d4/pay-api/src/pay_api/services/receipt.py#L198C33-L198C33
it's from database, should be auto-updated when the database ready.

@ochiu
Copy link
Collaborator

ochiu commented Oct 17, 2023

bcgov/entity#17382

yes, but the payment method text is from api request payload body, I need to update other repo which send the request I think.

Got it! thanks for explaining.

@Jxio Jxio merged commit b0e8e57 into bcgov:main Oct 17, 2023
7 checks passed
AbrahamRostampoor pushed a commit to AbrahamRostampoor/sbc-pay that referenced this pull request Oct 26, 2023
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