Closed
Bug 1029433
Opened 10 years ago
Closed 10 years ago
Desktop Client needs in-call remote party identification display
Categories
(Hello (Loop) :: Client, enhancement, P2)
Hello (Loop)
Client
Tracking
(firefox33 unaffected, firefox34+ fixed, firefox35 verified)
Tracking | Status | |
---|---|---|
firefox33 | --- | unaffected |
firefox34 | + | fixed |
firefox35 | --- | verified |
People
(Reporter: standard8, Assigned: standard8)
References
()
Details
(Whiteboard: [loop-inccall1])
User Story
This should be done after bug 1020449 which will get the required information into the conversation window. During a call, display the user-specified identifier in the title bar. If there is no identifier, then display the link.
Attachments
(1 file)
4.77 KB,
patch
|
NiKo
:
review+
lmandel
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•10 years ago
|
||
Darrin, can you check the user story here? I'm wondering if rather than displaying the full link we should just display the token part (or something else entirely).
User Story: (updated)
Flags: needinfo?(dhenein)
Comment 2•10 years ago
|
||
I think in the abssence of a username or link label, we should show the whole link. Removing the domain and just showing the token would be confusing, esp. without custom URLs (i.e. imagine seeing only 8zz2u89fh9 in your status bar?) Does that help?
Flags: needinfo?(dhenein)
Assignee | ||
Comment 3•10 years ago
|
||
Yes, thanks.
Updated•10 years ago
|
Whiteboard: p=1
Updated•10 years ago
|
Target Milestone: mozilla33 → mozilla34
Updated•10 years ago
|
Target Milestone: mozilla34 → mozilla35
Assignee | ||
Updated•10 years ago
|
Assignee | ||
Updated•10 years ago
|
Whiteboard: [loop-inccall1]
Comment 4•10 years ago
|
||
Based on work for Fx35 already - this is definitely below the bar. with 1:1 calling - in call party identification is lower priority (calling party identification on incoming call notification is in Fx34).
Priority: P1 → P2
Target Milestone: mozilla35 → mozilla36
Assignee | ||
Comment 5•10 years ago
|
||
Simple patch to change the title bar when in-call. This uses the work in bug 1020449. I've improved the tests a bit, and extended the test on the model to include checking for an empty string if we really can't work it out. For direct calls the information won't show up until bug 1079950 gets deployed on the server.
Attachment #8503080 -
Flags: review?(nperriault)
Assignee | ||
Comment 6•10 years ago
|
||
Pulling back into 35 - easy half hour fix.
Assignee: nobody → standard8
Target Milestone: mozilla36 → mozilla35
Comment on attachment 8503080 [details] [diff] [review] When in a Loop call, the title bar should display the remote party's information. Review of attachment 8503080 [details] [diff] [review]: ----------------------------------------------------------------- LGTM, r=me. ::: browser/components/loop/content/js/conversation.js @@ +234,5 @@ > ) > ); > } > case "connected": { > + document.title = this.props.conversation.getCallIdentifier(); Makes me think generic document title handling could be handled in mixin, a bit along the lines of what's been suggested in https://bugzilla.mozilla.org/show_bug.cgi?id=1078718#c3 - which would be a bit more easily/cleanly testable. I've just filed bug 1081079 about this.
Attachment #8503080 -
Flags: review?(nperriault) → review+
Assignee | ||
Comment 8•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/16582da55135
https://hg.mozilla.org/mozilla-central/rev/16582da55135
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 10•10 years ago
|
||
Comment on attachment 8503080 [details] [diff] [review] When in a Loop call, the title bar should display the remote party's information. Approval Request Comment [Feature/regressing bug #]: Part of the caller Id work (extension of bug 1020449) [User impact if declined]: Whilst in a call, the callee would have "Conversation Request" rather than the id of the caller. [Describe test coverage new/current, TBPL]: Landed on central, has unit tests [Risks and why]: Small patch that only changes the document title. [String/UUID change made/needed]: None
Attachment #8503080 -
Flags: approval-mozilla-beta?
Assignee | ||
Comment 11•10 years ago
|
||
QA: This is for direct calls only, and will only work 100% when bug 1079950 is deployed to production. In the meantime this can be tested against the development server, See bug 1020449 comment 32 for how to set up and test this.
Comment 12•10 years ago
|
||
Hi Pauly, Can you test this together with bug 1020449? You should be able to test this with Nightly or Aurora. Special instructions are in bug 1020449 since we need to use the loop dev server until bug 1079950 is deployed to production (hopefully later this week).
Flags: needinfo?(paul.silaghi)
Comment 13•10 years ago
|
||
Email/URL for registered users/guests displayed in the title bar. Verified fixed FF 35.0a2 (2014-10-15) Win 7, OS X 10.9.5
Comment 14•10 years ago
|
||
Comment on attachment 8503080 [details] [diff] [review] When in a Loop call, the title bar should display the remote party's information. Beta+
Attachment #8503080 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•10 years ago
|
Comment 16•10 years ago
|
||
I'm deprioritizing this for further QE testing as this has already been verified in Aurora. We've got more important bugs that need testing right now but may loop back around to this if time allows.
Flags: qe-verify-
Flags: qe-verify+
Flags: in-testsuite?
Updated•10 years ago
|
Iteration: --- → 35.3
Assignee | ||
Comment 17•9 years ago
|
||
Clearing in-testsuite requests for features that are being removed from Hello as part of te user journey work in bug 1209713.
Flags: in-testsuite?
You need to log in
before you can comment on or make changes to this bug.
Description
•