Skip to content

feat: Check if NFC is enabled and prompt user to enable it#4482

Merged
jamesarich merged 1 commit into
mainfrom
feat/nfc
Feb 6, 2026
Merged

feat: Check if NFC is enabled and prompt user to enable it#4482
jamesarich merged 1 commit into
mainfrom
feat/nfc

Conversation

@jamesarich

Copy link
Copy Markdown
Collaborator
  • Add a check to see if NFC is enabled before starting a scan.
  • Implement a dialog to inform the user that NFC is disabled, with a button to open system settings.
  • Add nfc_disabled, open_settings and related string resources.
  • Create an openNfcSettings context extension for easy access to NFC settings.
  • Refactor MainActivity to use a common handleMeshtasticUri function for both deep links and NFC NDEF messages.

- Add a check to see if NFC is enabled before starting a scan.
- Implement a dialog to inform the user that NFC is disabled, with a button to open system settings.
- Add `nfc_disabled`, `open_settings` and related string resources.
- Create an `openNfcSettings` context extension for easy access to NFC settings.
- Refactor `MainActivity` to use a common `handleMeshtasticUri` function for both deep links and NFC NDEF messages.
@github-actions github-actions Bot added the enhancement New feature or request label Feb 6, 2026
@jamesarich jamesarich enabled auto-merge February 6, 2026 17:54
@codecov

codecov Bot commented Feb 6, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 9.21%. Comparing base (935cff8) to head (ad8fbe4).
⚠️ Report is 3 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...tlin/org/meshtastic/core/ui/component/ImportFab.kt 0.00% 4 Missing ⚠️
.../settings/radio/component/NetworkConfigItemList.kt 0.00% 4 Missing ⚠️
...n/org/meshtastic/core/ui/util/ContextExtensions.kt 0.00% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##            main   #4482      +/-   ##
========================================
- Coverage   9.22%   9.21%   -0.01%     
========================================
  Files        427     427              
  Lines      14332   14342      +10     
  Branches    2385    2385              
========================================
  Hits        1322    1322              
- Misses     12757   12767      +10     
  Partials     253     253              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jamesarich jamesarich added this pull request to the merge queue Feb 6, 2026
Merged via the queue into main with commit 1e255a5 Feb 6, 2026
8 of 9 checks passed
@jamesarich jamesarich deleted the feat/nfc branch February 6, 2026 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant