Skip to content

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

Open
CrisTofani wants to merge 11 commits intomasterfrom
IOPLT-1572
Open

feat: [IOPLT-1572] Adds expo-modules support and install expo-screen-capture#7840
CrisTofani wants to merge 11 commits intomasterfrom
IOPLT-1572

Conversation

@CrisTofani
Copy link
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
Contributor

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

@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2026

Jira Pull Request Link

This Pull Request refers to Jira issues:

@github-actions
Copy link
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

codecov bot commented Feb 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.47%. Comparing base (d2a897c) to head (022fc78).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #7840   +/-   ##
=======================================
  Coverage   60.47%   60.47%           
=======================================
  Files        1942     1942           
  Lines       42840    42840           
  Branches     9823     9900   +77     
=======================================
  Hits        25907    25907           
+ Misses      16854    16843   -11     
- Partials       79       90   +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 d2a897c...022fc78. 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.

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.

4 participants