feat: [IOBP-687,IOBP-886] Add the recent used payment method when the user pays #415
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #415 +/- ##
==========================================
- Coverage 64.38% 64.34% -0.05%
==========================================
Files 167 167
Lines 5332 5354 +22
Branches 719 748 +29
==========================================
+ Hits 3433 3445 +12
+ Misses 1898 1769 -129
- Partials 1 140 +139
... and 34 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Hantex9
added a commit
to pagopa/io-app
that referenced
this pull request
Oct 14, 2024
…ayment flow (#6234) ## Short description This PR implements the ability to display the most recently used payment method during in-app payments, ensuring it matches the method used in a previous transaction. ## List of changes proposed in this pull request - Updated the open API definitions; - Added saga, action & reducer about the `recentUsedPaymentMethod` - Added an additional list item at the top of the list that contains the most recently used payment method if available, otherwise it isn't showed - Added a logic to remove the recently used payment method from the cluster membership ## How to test - Checkout this PR: pagopa/io-dev-api-server#415 from the io-dev-api-server and generate the API definitions; - Start the io-dev-api-server; - With the app in local env, start a payment flow from the "Payments" section screen tapping the CTA "Paga un avviso" - Then tap on the CTA "Digita" - Inside of it, type any notice code -> go forward and type any fiscal code; - After, you should be able to see a list of available payment method, complete the first payment choosing any payment method; - When you complete the transaction, try to start another payment flow, you should be able to see the payment method previously you choose in the "Recently used" section; ## Preview https://github.com/user-attachments/assets/62c30602-9581-42cd-b75c-dd3c0703aa22 --------- Co-authored-by: Mario Perrotta <[email protected]>
hevelius
approved these changes
Oct 14, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Short description
This PR updates the ecommerce and pagopa wallet APIs definitions to handle even the recent used payment method when the user pays.
List of changes proposed in this pull request
/user/lastPaymentMethodUsed;POST /transactions/:transactionId/auth-requestsis called;How to test