Add originator information to callback

[vicdelfant]

Is your feature request related to a problem? Please describe.
No

Describe the solution you'd like
Currently, the startReadSMS callback only receives the message body and not the sender's number. Having the sender information would make this library a lot more valuable because it can then be used to fully handle inbound text messages.

Describe alternatives you've considered
None available, this library seems to be one of a kind in the Expo ecosystem :)

Additional context
The getMessageFromMessageIntent function only takes the getDisplayMessageBody():

react-native-expo-read-sms/android/src/main/java/com/reactlibrary/RNExpoReadSmsModule.java

Line 82 in ceb1eac

message = currentMessage.getDisplayMessageBody();

Using a structure for the message and returning the getDisplayOriginatingAddress could solve this, but I'm by no means enough of an Android developer to test this… unfortunately.

[maniac-tech]

Thank you for raising the issue with all the relevant details @vicdelfant .

Let me check this and get back to you in ~2 Days

[maniac-tech]

Did a small POC using the getDisplayOriginatingAddress, and is working as expected.

Will create a branch and share a WIP PR by tomorrow EOD, that being said we may expect a new version to be made available by Monday post testing these changes for any potential issues.

[vicdelfant]

Nice! Thank you 🙏 Ping me here once the PR is ready and I’d be happy to test too.

[maniac-tech]

Nice! Thank you 🙏 Ping me here once the PR is ready and I’d be happy to test too.

Quick question: Are you currently using this ?

[vicdelfant]

We actually are, yes. It's for an in-house app but it'll be an integral part of the operational side of things :)

[maniac-tech]

Oh, happy to hear that the library is helping solve a use case :D

[maniac-tech]

This has been fixed in the PR #26 , and the npm package has been deployed with a new version 1.1 .

[vicdelfant]

Great, thank you! 🙏