-
Notifications
You must be signed in to change notification settings - Fork 104
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
chore: [IOBP-616] Add missing parameters to payment's calculateFees
request payload
#5667
chore: [IOBP-616] Add missing parameters to payment's calculateFees
request payload
#5667
Conversation
calculateFees
request payloadcalculateFees
request payload
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #5667 +/- ##
==========================================
- Coverage 48.31% 48.31% -0.01%
==========================================
Files 1466 1466
Lines 31114 31118 +4
Branches 7559 7559
==========================================
Hits 15034 15034
- Misses 16011 16015 +4
Partials 69 69
Continue to review full report in Codecov by Sentry.
|
…ncano-alcuni-campi-nella-richiesta-calculate-fees
9cc4c6e
to
82ffe1a
Compare
…ncano-alcuni-campi-nella-richiesta-calculate-fees
…ncano-alcuni-campi-nella-richiesta-calculate-fees
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.
LGTM, PSP list shows up as expected.
… request payload (#5667) ## Short description This PR adds `isAllCCP` and `primaryCreditorInstitution` parameters to the payment's `calculateFees` request payload ## List of changes proposed in this pull request - Updated pagoPA payments and wallet API definitions - Added `isAllCCP` and `primaryCreditorInstitution` to calculateFees request payload ## How to test In UAT environment, try to make a payment. You should be able to see the PSP list --------- Co-authored-by: Mario Perrotta <mario.perrotta@pagopa.it> Co-authored-by: Martino Cesari Tomba <60693085+forrest57@users.noreply.github.com>
… request payload (#5667) ## Short description This PR adds `isAllCCP` and `primaryCreditorInstitution` parameters to the payment's `calculateFees` request payload ## List of changes proposed in this pull request - Updated pagoPA payments and wallet API definitions - Added `isAllCCP` and `primaryCreditorInstitution` to calculateFees request payload ## How to test In UAT environment, try to make a payment. You should be able to see the PSP list --------- Co-authored-by: Mario Perrotta <mario.perrotta@pagopa.it> Co-authored-by: Martino Cesari Tomba <60693085+forrest57@users.noreply.github.com>
Short description
This PR adds
isAllCCP
andprimaryCreditorInstitution
parameters to the payment'scalculateFees
request payloadList of changes proposed in this pull request
isAllCCP
andprimaryCreditorInstitution
to calculateFees request payloadHow to test
In UAT environment, try to make a payment. You should be able to see the PSP list