Closed
Bug 1062595
Opened 9 years ago
Closed 9 years ago
No confirm window pops up to inform users that deleted numbers are removed from an ICE contact.
Categories
(Firefox OS Graveyard :: Gaia::Contacts, defect)
Tracking
(tracking-b2g:backlog, b2g-v2.1 affected, b2g-v2.2 verified)
People
(Reporter: oteo, Assigned: jmcf)
References
Details
(Keywords: late-l10n, Whiteboard: [p=1] [2.1-flame-test-run-2])
Attachments
(2 files)
191 bytes,
text/html
|
arcturus
:
review+
arcturus
:
feedback+
fabrice
:
approval-gaia-v2.1-
|
Details |
2.30 MB,
application/pdf
|
Details |
According to the ICE spec (https://bug1026682.bugzilla.mozilla.org/attachment.cgi?id=8459474 on page 15) when number/s is/are deleted from an ICE contact, user should be informed with a confirm window pop-up. Flame 2.1 Gecko-6be7031 Gaia-e1f6d88 Pre-requisites: 1. ICE Contact saved Steps to reproduce: 1.Go to contacts app and select an ICE contact from the list 2.Tap to edit contact 3.Remove a number or several numbers from the Contact Expected results: A confirm window should inform to the user that the deleted number are removed from the ICE list as well Actual results: no Confirm window implemented
Reporter | ||
Comment 1•9 years ago
|
||
[Blocking Requested - why for this release]: It's really important for the user be aware that deleting those numbers can make impossible to call to those preferred contacts in case of emergency
Blocks: 1026682
blocking-b2g: --- → 2.1?
Comment 3•9 years ago
|
||
Carrie, what should we say if we remove all the phones for a contact? Same wording?
Flags: needinfo?(cawang)
Comment 4•9 years ago
|
||
Hi, If the ICE contact has more than one number and we only delete one of them, we show the strings on p.15 Screen 3 and the contact will not be removed from ICE list. If all the numbers of the ICE contact are deleted the contact will be removed from ICE contacts and we show the strings as below, ---------------------------------------------- All the numbers are deleted. This contact is removed from the ICE Contacts. I'll add this case to the spec to keep it updated. Thanks!
Flags: needinfo?(cawang)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → jmcf
Comment 6•9 years ago
|
||
Triage group discussed and this doesn't meet the blocking criteria for stopping a whole release. Please review the criteria at https://wiki.mozilla.org/B2G/Triage#Blocker_Triage_Guidelines. This is a new feature, and the lack of this confirmation prompt doesn't prevent the use of the feature. If this is fixed in 2.1 timeframe, ask for approval and land! Otherwise, it seems like it can ship in the next version. Wesley and Maria, if there's some reason this should actually stop the release from shipping, please discuss!
blocking-b2g: 2.1+ → 2.1?
Updated•9 years ago
|
blocking-b2g: 2.1? → backlog
Comment 7•9 years ago
|
||
This will have an easy fix and the improve to the user experience will be significant, so let's fix this and ask for approval
Assignee | ||
Comment 8•9 years ago
|
||
Francisco, Please could you do a quick review? Then, I can polish it up and create a test
Attachment #8488563 -
Flags: feedback?(francisco)
Comment 9•9 years ago
|
||
Comment on attachment 8488563 [details]
24000.html
Definitely looking pretty good
Attachment #8488563 -
Flags: feedback?(francisco) → feedback+
Assignee | ||
Comment 10•9 years ago
|
||
Comment on attachment 8488563 [details] 24000.html Francisco, I believe this is ready to be reviewed. There are no tests in the PR because we are going to create an integration test for this (bug 1067234).
Attachment #8488563 -
Flags: review?(francisco)
Assignee | ||
Updated•9 years ago
|
Target Milestone: --- → 2.1 S5 (26sep)
Updated•9 years ago
|
Whiteboard: [p=1]
Comment 11•9 years ago
|
||
Comment on attachment 8488563 [details]
24000.html
We are not removing from ICE the contact if we remove all the phone numbers, we are just displaying the message (which looks good).
Attachment #8488563 -
Flags: review?(francisco) → review-
Assignee | ||
Comment 12•9 years ago
|
||
Francisco, The purpose of the bug is to show the alert to the user but not to actually remove the ICE contact. That should be another bug, and I believed you were implementing that. If that is not the case I can open a new one and solve it. thanks
Flags: needinfo?(francisco)
Assignee | ||
Updated•9 years ago
|
Attachment #8488563 -
Flags: review- → review?(francisco)
Comment 13•9 years ago
|
||
(In reply to Jose Manuel Cantera from comment #12) > Francisco, > > The purpose of the bug is to show the alert to the user but not to actually > remove the ICE contact. That should be another bug, and I believed you were > implementing that. > > If that is not the case I can open a new one and solve it. > > thanks The previous work done was about delete contacts (full contacts), and removing the ICE group. If we perform a check about removing just the phone, if we remove all the phones, we should remove that contact from the ICE group as well.
Flags: needinfo?(francisco)
Comment 14•9 years ago
|
||
Sorry, when I mean the ice group, I mean removing them as ICE contact. Then the group will dissapear if it was the only ICE contact.
Assignee | ||
Comment 15•9 years ago
|
||
(In reply to Francisco Jordano [:arcturus] [:francisco] from comment #14) > Sorry, when I mean the ice group, I mean removing them as ICE contact. > > Then the group will dissapear if it was the only ICE contact. Yes, but that should be solved in another bug. Can we land this and then open a new bug for that functionality?
Comment 16•9 years ago
|
||
Fair enough, we do have bug 1061420 to actually perform that last bit. Please, go ahead and merge once everything is green.
Updated•9 years ago
|
Attachment #8488563 -
Flags: review?(francisco) → review+
Updated•9 years ago
|
QA Whiteboard: [QAnalyst-Triage?]
status-b2g-v2.1:
--- → affected
status-b2g-v2.2:
--- → affected
Flags: needinfo?(ktucker)
Whiteboard: [p=1] → [p=1] [2.1-flame-test-run-2]
Updated•9 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Assignee | ||
Comment 17•9 years ago
|
||
landed in master: https://github.com/mozilla-b2g/gaia/commit/0d11eae32af28284ab9c4520d45af4e12dcd624f
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•9 years ago
|
Keywords: verifyme
Whiteboard: [p=1] [2.1-flame-test-run-2] → [p=1] [2.1-flame-test-run-2]
Comment 18•9 years ago
|
||
Comment on attachment 8488563 [details]
24000.html
[Approval Request Comment]
[Bug caused by] (feature/regressing bug #):
Ice contacts feature
[User impact] if declined:
User won't know that could be delete information that later could be use in case of an emergency
[Testing completed]:
Smoke test passed on flame
[Risk to taking this patch] (and alternatives if risky):
Pretty low risk
[String changes made]:
ICEContactDelTel
ICEContactDelTelAll
Attachment #8488563 -
Flags: approval-gaia-v2.1?(bbajaj)
Comment 19•9 years ago
|
||
Tested and *not* working, because if i set an ICE contact with 3 phones numbers only I can see warning when I delete only first phone number. If i delete a phone number, I can see warning, but if i close contact application and after i open app and delete other phone number to ICE contact I can't see warning. Flame 2.2 Gecko: cce3dff Gaia: 897bd62
Comment 20•9 years ago
|
||
(In reply to Loli (:lolimartinezcr) from comment #19) > Tested and *not* working, because if i set an ICE contact with 3 phones > numbers only I can see warning when I delete only first phone number. If i > delete a phone number, I can see warning, but if i close contact application > and after i open app and delete other phone number to ICE contact I can't > see warning. > > Flame > 2.2 > Gecko: cce3dff > Gaia: 897bd62 Sorry but I have confused because when i tap in "X" button in phone number doesn't appear warning, it only appears when user presses "update" button. Tested and working Flame 2.2 Gecko: cce3dff Gaia: 897bd62
Comment 21•9 years ago
|
||
Comment on attachment 8488563 [details]
24000.html
Too late for new strings.
Attachment #8488563 -
Flags: approval-gaia-v2.1?(bbajaj) → approval-gaia-v2.1-
Reporter | ||
Updated•9 years ago
|
Comment 22•9 years ago
|
||
Update the spec as a reference. Thanks!
Comment 23•9 years ago
|
||
Issue is verified fixed in Flame 2.2 build (Full Flash, nightly). Actual Results: Assigning an ICE contact (Person A), then deleting their number(s) prompts the notification "Deleted phone numbers are removed from ICE Contact". Device: Flame Master Build ID: 20141015040201 Gaia: 5f1f0960ae9d22acf2a324ad37a48174d6df87f6 Gecko: 62f0b771583c Version: 36.0a1 (Master) Firmware Version: v180 User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Keywords: verifyme
Updated•9 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Updated•9 years ago
|
blocking-b2g: backlog → ---
tracking-b2g:
--- → backlog
You need to log in
before you can comment on or make changes to this bug.
Description
•