io-app icon indicating copy to clipboard operation
io-app copied to clipboard

feat: [IOPLT-724] Split up the backend status for better handling

Open CrisTofani opened this issue 1 year ago • 2 comments
trafficstars

Short description

This PR aims to split the backendStatus reducer and store section into 4 different reducers

List of changes proposed in this pull request

  • Removed backendStatus reducer
  • Created remoteConfig, sectionStatus, backendInfo, statusMessages reducers

How to test

Anything should work properly as usual after changes.

CrisTofani avatar Oct 17 '24 10:10 CrisTofani

Affected stories

  • 🌟 IOPLT-724: Come DEV voglio suddividere la gestione delle configurazioni remote sull'APP per ottimizzarne le evoluzioni future

Generated by :no_entry_sign: dangerJS against 9113772357447bfc88ea7451e17239cf315f4834

pagopa-github-bot avatar Oct 17 '24 10:10 pagopa-github-bot

Codecov Report

Attention: Patch coverage is 57.36842% with 81 lines in your changes missing coverage. Please review.

Project coverage is 46.98%. Comparing base (4f204b4) to head (9113772). Report is 674 commits behind head on master.

Files with missing lines Patch % Lines
ts/store/reducers/backendStatus/remoteConfig.ts 46.61% 61 Missing and 2 partials :warning:
ts/store/reducers/backendStatus/backendInfo.ts 64.70% 6 Missing :warning:
ts/store/reducers/backendStatus/statusMessages.ts 68.75% 5 Missing :warning:
ts/sagas/backendStatus.ts 0.00% 2 Missing :warning:
ts/screens/modal/SystemOffModal.tsx 0.00% 2 Missing :warning:
ts/store/reducers/backendStatus/sectionStatus.ts 93.10% 2 Missing :warning:
ts/components/wallet/PaymentMethodsList.tsx 0.00% 1 Missing :warning:
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6296      +/-   ##
==========================================
- Coverage   48.42%   46.98%   -1.45%     
==========================================
  Files        1488     1821     +333     
  Lines       31617    36949    +5332     
  Branches     7669     8799    +1130     
==========================================
+ Hits        15311    17360    +2049     
- Misses      16238    19533    +3295     
+ Partials       68       56      -12     
Files with missing lines Coverage Δ
ts/components/SectionStatus/index.tsx 100.00% <ø> (+2.77%) :arrow_up:
ts/components/SectionStatus/modal/index.tsx 61.70% <ø> (ø)
ts/components/screens/BaseHeader.tsx 78.37% <ø> (-5.63%) :arrow_down:
...s/components/screens/BaseScreenComponent/index.tsx 62.96% <ø> (-11.12%) :arrow_down:
...code/components/BarcodeScanBaseScreenComponent.tsx 77.77% <ø> (-1.59%) :arrow_down:
ts/features/barcode/screens/BarcodeScanScreen.tsx 1.38% <ø> (-0.07%) :arrow_down:
ts/features/bonus/cgn/screens/CgnDetailScreen.tsx 4.54% <ø> (-9.41%) :arrow_down:
...screens/activation/CgnCTAStartActivationScreen.tsx 16.00% <ø> (-6.73%) :arrow_down:
...ures/bonus/common/screens/AvailableBonusScreen.tsx 7.69% <ø> (-1.24%) :arrow_down:
ts/features/cieLogin/store/selectors/index.ts 66.66% <ø> (+16.66%) :arrow_up:
... and 87 more

... and 1360 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 0881665...9113772. Read the comment docs.

codecov[bot] avatar Oct 17 '24 15:10 codecov[bot]