Skip to content

Conversation

@naveensingh
Copy link
Member

@naveensingh naveensingh commented Dec 20, 2025

Type of change(s)

  • Bug fix
  • Feature / enhancement
  • Infrastructure / tooling (CI, build, deps, tests)
  • Documentation

What changed and why

  • Added goAsync() call before starting background work.
  • Switched to SimpleContactHelper.existsSync() method from feat: add sync method for checking known contacts commons#259 for a faster lookup instead of iterating over all contacts.
  • Cleaned up the receiver code and simplified threading logic.
  • MmsReceiver needs more changes, both here and in the mmslib for this fix.

Tests performed

Tested receiving and sending SMS

Closes the following issue(s)

Checklist

  • I read the contribution guidelines.
  • I manually tested my changes on device/emulator (if applicable).
  • I updated the "Unreleased" section in CHANGELOG.md (if applicable).
  • I have self-reviewed my pull request (no typos, formatting errors, etc.).
  • All checks are passing.

@naveensingh naveensingh merged commit 1e3f5e5 into main Dec 21, 2025
7 checks passed
@naveensingh naveensingh deleted the naveensingh/fix-sms-receiver-logic branch December 21, 2025 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sometimes no notifications for new received SMS messages

2 participants