Closed
Bug 1081154
Opened 10 years ago
Closed 10 years ago
Loop direct calls should attempt to call phone numbers as well as email addresses
Categories
(Hello (Loop) :: Client, defect)
Hello (Loop)
Client
Tracking
(firefox33 unaffected, firefox34+ verified, firefox35+ verified, firefox36 verified)
backlog | Fx34+ |
People
(Reporter: standard8, Assigned: standard8)
Details
(Whiteboard: [loop-uplift][loop-outcall1])
Attachments
(1 file, 1 obsolete file)
4.87 KB,
patch
|
mikedeboer
:
review+
lmandel
:
approval-mozilla-aurora+
lmandel
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
Currently, when calling a contact we only use any email addresses for a contact - in the list of ids we send to the server. We need to additionally include any phone number information that we have - import contacts have phone numbers already. When forming the list of phone numbers we should strip non-numbers & and leave a plus at the start.
Assignee | ||
Updated•10 years ago
|
Target Milestone: --- → mozilla35
Updated•10 years ago
|
backlog: --- → Fx34+
Comment 1•10 years ago
|
||
Mark -- I believe we need to prioritize this one bug over the standalone work if we want to get it into Fx34. My understanding is that this would be a relatively small patch. Is this something we could have soon?
Flags: needinfo?(standard8)
Assignee | ||
Comment 2•10 years ago
|
||
Simple patch that adds in phone numbers to the list of ids that are attempted to be called for a contact. Tested on desktop with emails, and on FxOS with the Hello app signed in with a phone number.
Attachment #8506050 -
Flags: review?(mdeboer)
Assignee | ||
Updated•10 years ago
|
Flags: needinfo?(standard8)
Assignee | ||
Comment 3•10 years ago
|
||
Updated patch to include stripping of unnecessary characters from phone numbers.
Attachment #8506050 -
Attachment is obsolete: true
Attachment #8506050 -
Flags: review?(mdeboer)
Attachment #8506080 -
Flags: review?(mdeboer)
Comment 4•10 years ago
|
||
Comment on attachment 8506080 [details] [diff] [review] Loop direct calls should attempt to call phone numbers as well as email addresses. Review of attachment 8506080 [details] [diff] [review]: ----------------------------------------------------------------- /me like! Thanks Mark.
Attachment #8506080 -
Flags: review?(mdeboer) → review+
Assignee | ||
Comment 5•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/2e896ba5d24f
Iteration: --- → 36.1
Points: --- → 2
Target Milestone: mozilla35 → mozilla36
Assignee | ||
Comment 6•10 years ago
|
||
Comment on attachment 8506080 [details] [diff] [review] Loop direct calls should attempt to call phone numbers as well as email addresses. Approval Request Comment [Feature/regressing bug #]: Part of call handling for Firefox Hello [User impact if declined]: Users won't be able to call Firefox OS mobiles that are signed into the Hello app using phone numbers. [Describe test coverage new/current, TBPL]: Tested locally, has unit tests to cover additions, landed on fx-team, should merge to nightly soon. [Risks and why]: Low, existing tests cover making sure the email addresses are still callable, extension for phone numbers also tested. [String/UUID change made/needed]: None
Attachment #8506080 -
Flags: approval-mozilla-beta?
Attachment #8506080 -
Flags: approval-mozilla-aurora?
Comment 7•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/2e896ba5d24f
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 8•10 years ago
|
||
[Tracking Requested - why for this release]: Tracking for release, See Comment 6
tracking-firefox34:
--- → ?
Comment 10•10 years ago
|
||
Untracking for QE verification since this appears to have pretty good test coverage already. Please needinfo me to request manual verification.
Flags: qe-verify-
Updated•10 years ago
|
status-firefox33:
--- → unaffected
status-firefox34:
--- → affected
status-firefox35:
--- → affected
status-firefox36:
--- → fixed
tracking-firefox35:
--- → +
Comment 11•10 years ago
|
||
Tested and works on Nightly 10/18; Windows and Linux
Comment 12•10 years ago
|
||
Also note I called a Hello app on Flame using it
Comment 13•10 years ago
|
||
Comment on attachment 8506080 [details] [diff] [review] Loop direct calls should attempt to call phone numbers as well as email addresses. Thanks for testing jesup. Approved for Aurora. If everything goes well with your testing on Aurora on Sunday, we'll get this uplifted for beta2.
Attachment #8506080 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Updated•10 years ago
|
Updated•10 years ago
|
Comment 15•10 years ago
|
||
Tested and good on Aurora nightly build; windows and linux.
Comment 17•10 years ago
|
||
Comment on attachment 8506080 [details] [diff] [review] Loop direct calls should attempt to call phone numbers as well as email addresses. Previously approved offline. Adding approval to the bug.
Attachment #8506080 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 18•10 years ago
|
||
Changing this to qe-verify+ since it's a blocker for release. Florin, please make sure this gets tested as part of Beta 2, assuming it uplifts in time.
Flags: qe-verify-
Flags: qe-verify+
Flags: needinfo?(florin.mezei)
Comment 19•10 years ago
|
||
(In reply to Anthony Hughes, QA Mentor (:ashughes) from comment #18) > Florin, please make sure this gets tested as part of Beta 2, assuming it > uplifts in time. I was blocked in testing this on Fx OS 2.0/188 - couldn't pass over the first screen in the Hello app. Nothing happens when I click on the 'use phone number' button, just a short "authenticating".
Comment 20•10 years ago
|
||
(In reply to Paul Silaghi, QA [:pauly] from comment #19) > I was blocked in testing this on Fx OS 2.0/188 - couldn't pass over the > first screen in the Hello app. Nothing happens when I click on the > 'use phone number' button, just a short "authenticating". Tony, is this a known issue with the FxOS client?
Flags: needinfo?(tchung)
Comment 21•10 years ago
|
||
This is caused by Bug 1086880 in v188 Per Mariateo, installing *2* SIMs will get you past the bug. ferjm gave this workaround as well: adb root adb shell stop b2g adb remount adb shell rm -r /system/b2g/distribution/bundles adb shell start b2g ----- (if this doesn't work, do an "adb reboot" instead) I tried the workaround with b2, and was able to call the fxos phone by number
Comment 22•10 years ago
|
||
yep, what jesup said. that bug is now blocking 2.1. you can retry this against v180 and see if it works.
Flags: needinfo?(tchung)
Comment 23•10 years ago
|
||
(In reply to Randell Jesup [:jesup] from comment #21) > I tried the workaround with b2, and was able to call the fxos phone by number Marking verified in 34b2
You need to log in
before you can comment on or make changes to this bug.
Description
•