Skip to content

feat: Integrate @maniac-tech/react-native-expo-read-sms at 9.1.2 on Expo SDK 52#23

Open
maniac-tech wants to merge 5 commits intomasterfrom
sdk-52
Open

feat: Integrate @maniac-tech/react-native-expo-read-sms at 9.1.2 on Expo SDK 52#23
maniac-tech wants to merge 5 commits intomasterfrom
sdk-52

Conversation

@maniac-tech
Copy link
Copy Markdown
Owner

Summary

Verification branch for react-native-expo-read-sms PR #99 — confirms the library at 9.1.2 builds and runs correctly on Expo SDK 52 / React Native 0.76.


Changes

package.json

  • Upgraded @maniac-tech/react-native-expo-read-sms: 8.0.2-alpha^9.1.2
  • Upgraded react-native: 0.74.30.74.5
  • Upgraded expo-build-properties: ~0.12.3~0.12.5
  • Upgraded @babel/core: ^7.12.9^7.24.0
  • Removed: react-native-paper, react-native-web, react-dom, expo-updates
  • Added build:android script for EAS builds
  • Renamed package: exporeadsms-testexpo-read-sms-test-app

app.json

  • Fixed android.permissions — now correctly declares RECEIVE_SMS and READ_SMS (was empty [])
  • Fixed minSdkVersion: 3423 (the previous value blocked all Android 5.0–13 devices)
  • Removed OTA updates config (expo-updates plugin, runtimeVersion, extra.eas)
  • Removed iOS and web targets (Android-only test app)
  • Renamed app/slug: ExpoReadSMS-TestExpoReadSMS-TestApp

App.js

  • Full rewrite: replaced react-native-paper UI with a minimal plain React Native UI
  • Directly calls all four library APIs and logs results on-screen:
    • checkIfHasSMSPermission() — displays both permission flags
    • requestReadSMSPermission() — triggers system dialog, then re-checks status
    • startReadSMS(callback) — starts listener; logs received SMS body
    • stopReadSMS() — stops listener; confirms via log
  • Removed useApp hook dependency (previously coupled to old UI)

.gitignore

  • Added /android and /ios — generated native folders excluded from version control (generated via expo prebuild)

readme.md

  • Replaced SDK-specific docs with a general SDK compatibility table linking all four branches (sdk-51 through sdk-54)
  • Added unified getting-started steps applicable across all branches

Testing

Property Value
Expo SDK ~52.0.0
React Native 0.76
Library version ^9.1.2
New Architecture Optional (off by default for existing projects)
Android targetSdk 34 / minSdk 23

Verification Checklist

  • App builds without errors (npx expo prebuild + npx expo run:android succeed)
  • App launches on device/emulator
  • checkIfHasSMSPermission() — returns correct { hasReadSmsPermission, hasReceiveSmsPermission } object
  • requestReadSMSPermission() — system dialog appears; returns true when granted
  • startReadSMS(callback) — starts without error; callback fires with status === "success"
  • Incoming SMS received — callback delivers correct [phoneNumber, messageBody] values
  • stopReadSMS() — stops listener; no further callbacks after stopping
  • No NativeEventEmitter warnings in console

Related

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request good first issue Good for newcomers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant