Multiple recipients don't display correctly in Android

[levibotelho]

Not sure if this is a problem with Android or the lib, but figured I'd bring this up.

If I create an SMS to two recipients in Android, my screen looks like this:

However if I create an SMS to two recipients via the lib, I get this:

Notice how only the last recipient is shown. This is what the "People & Options" menu looks like:

It appears that both numbers are being added (hence the block text) but that only the last is taken into account (hence the "people in this conversation" list). I'm not testing this on an actual device yet so I don't know if it's just a display issue or if the SMS will actually only be sent to the second recipient, but at the very least this could be confusing for the user from a UX standpoint.

[levibotelho]

Looking a bit more deeply into this it might be due to the device I'm on (Pixel 1 emulator). Not sure if it just requires a comma separator or if it's simply not possible on this device. Can investigate further if you want help...

[tkporter]

Hi @levibotelho, were you able to find a potential solution or reason for this behavior?

[levibotelho]

No. Poked around a bit and might just have to live with it... might be a device-dependent thing...

[Gazfay]

Have the same problem with devices on android 8.

[Gazfay]

Check it - https://github.com/Gazfay/react-native-sms/blob/master/android/src/main/java/com/tkporter/sendsms/SendSMSModule.java

[dancherb]

@Gazfay Just saw that you hooray'd my post - I'm actually reporting that there's still a bug: when you send a message to multiple recipients, the "success" status doesn't go through and you just get "cancelled" when you return to the app. (I'll delete previous replies for clarity.)

[ashleypeacock]

Tried this on Nexus 6 and Pixel 2 XL emulator and it can't take multiple recipients.

String separator = ";";
                if(android.os.Build.MANUFACTURER.equalsIgnoreCase("Samsung")){
                    separator = ",";
                }

is the issue. To work the deliminator needs to be a "," instead of a semi-colon.

Code works fine on Nokia 7 device.

[tkporter]

@ashleypeacock nice find! Would you be able to open up a PR with that change?