io-app
io-app copied to clipboard
feat: [IOBP-863,IOBP-864] Removed legacy wallet & payments features
Short description
This PR removes all elements related to the old wallet, payment flow, and onboarding process. It includes the deletion of screens, components, sagas, reducers, and other elements associated with these deprecated features.
List of changes proposed in this pull request
- A lot of deletion and code removal ☠️
How to test
To test this PR, both the new payment flow and the onboarding flow must work correctly. Additionally, it's important to test a payment initiated from the Message screen. @forrest57 @Vangaorth, this PR cannot be merged until it is approved by the communication team.
| Warnings | |
|---|---|
| :warning: |
Multiple stories with different types are associated with this Pull request.
Only one tag will be added, following the order: |
Affected stories
- 🌟 IOBP-863: Rimozione flussi legacy di onboarding
subtask of
- ⚡ IOBP-862: Rimozione flusso legacy Portafoglio
- 🌟 IOBP-864: Rimozione flussi legacy di pagamento
subtask of
- ⚡ IOBP-862: Rimozione flusso legacy Portafoglio
Generated by :no_entry_sign: dangerJS against b3342eb56831232edd3adf9133c0b361f014b82a
@Hantex9 If it helps, I've just run npx knip to find all the unused components. Some of them are false positives (because they're currently in use), but I see a lot of new entries compared to the past. For example:
ts/components/PreferencesListItem.tsxts/components/ScreenHeader.tsxts/components/SelectCalendarModal.tsxts/components/screens/DarkLayout.tsxts/components/screens/EdgeBorderComponent.tsxand so on…
Give it a look.
PS: I'm glad this time has come ✌🏻
Codecov Report
Attention: Patch coverage is 7.85714% with 129 lines in your changes missing coverage. Please review.
Project coverage is 47.69%. Comparing base (
4f204b4) to head (b3342eb). Report is 728 commits behind head on master.
Additional details and impacted files
@@ Coverage Diff @@
## master #6210 +/- ##
==========================================
- Coverage 48.42% 47.69% -0.74%
==========================================
Files 1488 1609 +121
Lines 31617 32225 +608
Branches 7669 7426 -243
==========================================
+ Hits 15311 15370 +59
- Misses 16238 16802 +564
+ Partials 68 53 -15
| Files with missing lines | Coverage Δ | |
|---|---|---|
| ts/features/barcode/screens/BarcodeScanScreen.tsx | 1.42% <ø> (-0.03%) |
:arrow_down: |
| ts/features/design-system/core/DSLegacyAdvice.tsx | 50.00% <ø> (ø) |
|
| ts/features/design-system/core/DSListItems.tsx | 55.55% <ø> (+28.88%) |
:arrow_up: |
| ...components/MessageDetail/MessageDetailsPayment.tsx | 100.00% <ø> (ø) |
|
| ...ents/MessageDetail/MessageDetailsPaymentButton.tsx | 66.66% <ø> (-4.77%) |
:arrow_down: |
| ...nents/MessageDetail/MessageDetailsStickyFooter.tsx | 80.45% <ø> (-3.76%) |
:arrow_down: |
| ...es/components/MessageDetail/MessagePaymentItem.tsx | 85.48% <ø> (-7.74%) |
:arrow_down: |
| ...tures/messages/saga/handlePaymentUpdateRequests.ts | 3.70% <ø> (-0.47%) |
:arrow_down: |
| ts/features/messages/saga/index.ts | 0.00% <ø> (ø) |
|
| ts/features/messages/utils/index.ts | 91.42% <100.00%> (-8.58%) |
:arrow_down: |
| ... and 39 more |
... and 1606 files with indirect coverage changes
Continue to review full report in Codecov by Sentry.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update 5ce870d...b3342eb. Read the comment docs.
Great work, it works as expected! 🎊
What I suggest here is to remove some unused files in payments folder:
ts/features/payments/checkout/types/PaymentHistory.tsts/features/payments/checkout/types/WalletPaymentPspSortType.tsts/features/payments/common/components/PaymentCardBankLogo.tsxts/features/payments/home/components/PaymentsListItemTransaction.tsxUse
npx knipto double check them
That's cool! Addressed into d96e40440c527b3beda72cf471552318cbe001ff
This PR is scheduled to be merged on Monday 18th November 2024 🗓️