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

refactor: [IOBP-1115] Replace deprecated components and functions for IDPay #6586

Open
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

LeleDallas
Copy link
Contributor

@LeleDallas LeleDallas commented Jan 7, 2025

Short description

This PR is replacing legacy components for IDPay screens to align them with design standards

Warning

This PR depends on #6584

List of changes proposed in this pull request

  • Import FooterActions from design system
  • Replace RNavScreenWithLargeHeader with IOScrollViewWithLargeHeader
  • Replace GradientScrollView with IOScrollViewWithLargeHeader (thanks @dmnplb)
  • Replace jest deprecated functions (toBeCalledWith, toBeCalledTimes and toBeCalled)
  • Replace deprecated Layout of react-native-reanimated with LinearTransition
  • Remove back button in onboarding screens with IOScrollViewWithLargeHeader
  • Replace all scrollview with IOScrollViewWithLargeHeader

How to test

Ensure that all legacy components are replaced and properly aligned with standards

Preview

Old New
Screen.Recording.2025-01-08.at.17.49.58.mov
Screen.Recording.2025-01-08.at.17.13.07.mov
Screen.Recording.2025-01-08.at.17.44.15.mov
Screen.Recording.2025-01-08.at.17.13.44.mov
Screen.Recording.2025-01-08.at.17.53.58.mov
Screen.Recording.2025-01-08.at.17.28.48.mov
Screen.Recording.2025-01-08.at.17.52.13.mov
Screen.Recording.2025-01-08.at.17.29.27.mov

replace TopScreenComponent with useHeaderSecondLevel
replace BaseScreenComponent with useHeaderSecondLevel
Replace RNavScreenWithLargeHeader with IOScrollViewWithLargeHeader
replace Layout with LinearTransition
replace GradientScrollView with IOScrollView
@LeleDallas LeleDallas added the IO-Bonus e pagamenti IO - Bonus e pagamenti label Jan 7, 2025
@LeleDallas LeleDallas self-assigned this Jan 7, 2025
Copy link
Contributor

github-actions bot commented Jan 7, 2025

Jira Pull Request Link

This Pull Request refers to the following Jira issue IOBP-1115

Copy link

codecov bot commented Jan 7, 2025

Codecov Report

Attention: Patch coverage is 1.53846% with 64 lines in your changes missing coverage. Please review.

Project coverage is 41.43%. Comparing base (b207270) to head (4e9b4d1).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...pay/configuration/screens/IbanEnrollmentScreen.tsx 0.00% 11 Missing ⚠️
...nfiguration/screens/ConfigurationSuccessScreen.tsx 0.00% 8 Missing ⚠️
...pay/configuration/screens/IbanOnboardingScreen.tsx 0.00% 7 Missing ⚠️
...figuration/screens/InstrumentsEnrollmentScreen.tsx 0.00% 6 Missing ⚠️
.../profile/playgrounds/IdPayOnboardingPlayground.tsx 0.00% 6 Missing ⚠️
...boarding/screens/MultiValuePrerequisitesScreen.tsx 0.00% 5 Missing ⚠️
...uration/screens/IdPayDiscountInstrumentsScreen.tsx 0.00% 4 Missing ⚠️
...pay/onboarding/screens/PDNDPrerequisitesScreen.tsx 0.00% 4 Missing ⚠️
...nboarding/screens/BoolValuePrerequisitesScreen.tsx 0.00% 3 Missing ⚠️
...atures/idpay/code/screens/IdPayCodeRenewScreen.tsx 0.00% 2 Missing ⚠️
... and 6 more

❗ There is a different number of reports uploaded between BASE (b207270) and HEAD (4e9b4d1). Click for more details.

HEAD has 6 uploads less than BASE
Flag BASE (b207270) HEAD (4e9b4d1)
12 6
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6586      +/-   ##
==========================================
- Coverage   49.30%   41.43%   -7.88%     
==========================================
  Files        1565     1554      -11     
  Lines       32223    32042     -181     
  Branches     7288     7224      -64     
==========================================
- Hits        15887    13275    -2612     
- Misses      16298    18732    +2434     
+ Partials       38       35       -3     
Files with missing lines Coverage Δ
ts/features/idpay/barcode/navigation/navigator.tsx 66.66% <ø> (ø)
...idpay/barcode/screens/IdPayBarcodeResultScreen.tsx 15.15% <ø> (ø)
ts/features/idpay/code/navigation/navigator.tsx 66.66% <ø> (ø)
...ures/idpay/code/screens/IdPayCodeDisplayScreen.tsx 96.00% <ø> (ø)
...tures/idpay/configuration/navigation/navigator.tsx 37.50% <ø> (ø)
...ion/screens/InitiativeConfigurationIntroScreen.tsx 15.78% <ø> (+1.50%) ⬆️
...y/details/screens/IdPayInitiativeDetailsScreen.tsx 83.52% <100.00%> (+0.19%) ⬆️
...dpay/details/screens/IdPayOperationsListScreen.tsx 8.82% <ø> (ø)
...features/idpay/onboarding/navigation/navigator.tsx 37.50% <ø> (ø)
...pay/onboarding/screens/InitiativeDetailsScreen.tsx 9.09% <ø> (ø)
... and 16 more

... and 241 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 data
Powered by Codecov. Last update 4c8f237...4e9b4d1. Read the comment docs.

Replace IOScrollView with IOScrollViewWithLargeHeader as it provides more props for rendering purpose
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IO-Bonus e pagamenti IO - Bonus e pagamenti
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant