[Messages][SMS] Recipient removal page shown if it is not a contact, and the type is also missing from page.



Firefox OS
3 years ago
9 months ago


(Reporter: ericcc, Unassigned, Mentored)


Gonk (Firefox OS)

Firefox Tracking Flags

(b2g-v2.1 affected)


(Whiteboard: [sms-papercuts][lang=js][good first bug])


(1 attachment)



3 years ago
Created attachment 8490649 [details]

### STR
1. Have a previous call or sms to device from A
2. Have contact B stored in Contacts
3. New sms and add recipient A and B
4. Tap A, a recipient removal page show.

### Actual
If tapping on recipient of a number (who has never called or sms to testing device before), the page is not shown, and the number just becomes editable.

If the page shown is correct, there is a formatting problem, 'type' missing.

### Version
Gaia      47939f4c41d0c941e5047e5d1af74a79b7d8e0d5
Gecko     https://hg.mozilla.org/releases/mozilla-aurora/rev/e20869e87e23
BuildID   20140916160202
Version   34.0a2
ro.build.date=Fri Jun 27 15:57:58 CST 2014


3 years ago
QA Whiteboard: [COM=Gaia::SMS]
Flags: needinfo?(felash)
Hey Eric, is it possible that you do a video with this issue? I don't reproduce with your steps here...
Flags: needinfo?(felash) → needinfo?(echang)

Comment 2

3 years ago
Hi Julien, I updated the STR and here is the video.

### STR changed 
1. Have a previous sms sent to device from A (A not saved to Contacts)
2. Have contact B stored in Contacts
3. Create a new sms and add recipient A and B
4. Tap A, a recipient removal page show.

Flags: needinfo?(echang)
Ah yeah, now it makes sense, thanks !

NI Jenny about this, as I don't know if we should display the "contact" information for the "unknown contact" recipients, or just let the user edit them.

The fact that we display the "," could come from bug 1040271.
Flags: needinfo?(jelee)

Comment 4

3 years ago
There are 4 types of recipient number/name that will appear in the to field, listed below is the type and what is shown when tapping on the bubble:

1. Saved contact with name and number
[Contact name]
[Phone number type, phone number]
2. Saved contact with only number
[Phone number]
[Phone number type, phone number]
3. Number that the device received message from for at least one time
[Phone number]
[, phone number]
4. Number that's typed in manually
---> cursor appears, bubble gone

I think when user taps on a bubble in to field, the main purpose is most likely to remove it (or see detail) rather than edit it. So for type 3, is it possible to add "Unknown" in front of phone number so it would look like:

[Phone number]
["Unknown", phone number] 

Let me know what you think, thanks :)
Flags: needinfo?(jelee)
Mentor: felash@gmail.com
Whiteboard: [sms-papercuts][lang=js][good first bug]
Mass closing of Gaia::SMS bugs. End of an era :(
Last Resolved: 9 months ago
Resolution: --- → WONTFIX
Mass closing of Gaia::SMS bugs. End of an era :(
You need to log in before you can comment on or make changes to this bug.