Skip to content

feat: [IOPLT-1572] Adds expo-modules support and install expo-screen-capture#7840

Merged
CrisTofani merged 12 commits intomasterfrom
IOPLT-1572
Mar 19, 2026
Merged

feat: [IOPLT-1572] Adds expo-modules support and install expo-screen-capture#7840
CrisTofani merged 12 commits intomasterfrom
IOPLT-1572

Conversation

@CrisTofani
Copy link
Copy Markdown
Contributor

Note

This PR requires to elevate min iOS version to 15.1

Short description

This PR installs expo and fixes related dependencies to build on both iOS and Android.
Furthermore in this PR the first expo-module has been installed, replacing previous react-native module.
expo-screen-capture takes place of react-native-screenshot-prevent removing the related references and patch.

List of changes proposed in this pull request

  • Installs expo
  • Removes unsupported and unused dependencies
  • Disables the auto-linking on local env for @pagopa/react-native-cie on iOS
  • Replaces react-native-screenshot-prevent with expo-screen-capture

How to test

To test the PR follow these steps:

  • Remove node_modules runninng rm -rf node_modules
  • Remove Pods and derived data running cd ios; rm -rf Pods; rm -rf DerivedData
  • Run complete new installation of packages with yarn install
  • Run new pod installation command with yarn dev:pod-install
  • Build and compile iOS and Android project to check anything's working as usual

@LazyAfternoons
Copy link
Copy Markdown
Contributor

I think the README.md should be updated accordingly with the new pod install command.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 9, 2026

Jira Pull Request Link

This Pull Request refers to Jira issues:

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 9, 2026

PR Title Validation for conventional commit type

All good! PR title follows the conventional commit type.

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.26%. Comparing base (fd699f9) to head (49925ae).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #7840   +/-   ##
=======================================
  Coverage   60.26%   60.26%           
=======================================
  Files        1968     1968           
  Lines       43943    43943           
  Branches    10140    10141    +1     
=======================================
  Hits        26482    26482           
+ Misses      17383    17372   -11     
- Partials       78       89   +11     
Files with missing lines Coverage Δ
ts/utils/hooks/usePreventScreenCapture.ts 100.00% <100.00%> (ø)

... and 11 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 c53cd92...49925ae. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mastro993
Copy link
Copy Markdown
Contributor

mastro993 commented Feb 25, 2026

I’ve linked issue #7828 to this PR.

While investigating, I identified react-native-screenshot-prevent as the cause of the glitch. Since this PR removes that dependency, it should also resolve the issue.

Copy link
Copy Markdown
Contributor

@LeleDallas LeleDallas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested B&P flow with iOS and Android real devices and I didn't spot any regressions!

I also confirm that it solves #7828 as @mastro993 said

@CrisTofani CrisTofani added this pull request to the merge queue Mar 19, 2026
Merged via the queue into master with commit ac74742 Mar 19, 2026
23 checks passed
@CrisTofani CrisTofani deleted the IOPLT-1572 branch March 19, 2026 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Disallineamento tab bar dopo apertura documento

5 participants