Skip to content

refactor: [IOPLT-1583] Migrates to react-native-ble-plx - #8448

Open
LeleDallas wants to merge 7 commits into
masterfrom
IOPLT-1583
Open

refactor: [IOPLT-1583] Migrates to react-native-ble-plx#8448
LeleDallas wants to merge 7 commits into
masterfrom
IOPLT-1583

Conversation

@LeleDallas

Copy link
Copy Markdown
Contributor

Short description

This pull request migrates the codebase from using react-native-bluetooth-state-manager to react-native-ble-plx for checking Bluetooth state in the app.

List of changes proposed in this pull request

  • Replaced the react-native-bluetooth-state-manager dependency with react-native-ble-plx;
  • Apply react-native-ble-plxto check bluetooth activation;
  • Add Jest mock for react-native-ble-plx.

How to test

Play with bluetooth activation and permissions, and verify the expected outcomes through the proximity/ITW flow.

@github-actions

Copy link
Copy Markdown
Contributor

PR Title Validation for conventional commit type

All good! PR title follows the conventional commit type.

@github-actions

github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Jira Pull Request Link

This Pull Request refers to Jira issues:

@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.79%. Comparing base (9644de0) to head (079c89c).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...tures/itwallet/presentation/proximity/utils/ble.ts 33.33% 2 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #8448   +/-   ##
=======================================
  Coverage   63.78%   63.79%           
=======================================
  Files        1978     1978           
  Lines       37608    37609    +1     
  Branches     6360     6360           
=======================================
+ Hits        23989    23992    +3     
+ Misses      12242    12240    -2     
  Partials     1377     1377           
Files with missing lines Coverage Δ
...tures/itwallet/presentation/proximity/utils/ble.ts 20.00% <33.33%> (+4.21%) ⬆️

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0e6915e...079c89c. 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.

2 participants