Closed Bug 987597 Opened 10 years ago Closed 10 years ago

Adapt desktop client to send an identifier when obtaining a call-url


(Hello (Loop) :: Client, defect, P1)



(Not tracked)

backlog mlp+


(Reporter: standard8, Assigned: rgauthier)



(Whiteboard: [landed on loop-ui-initial][s=mlpnightly1, p=.25])

User Story

Approximate UX:

- Open panel
- Enter a nickname
- Click the get-url button


(1 file)

Following bug 986533 and various discussions, we need the link generator to specify a name/id for the link, so that the user knows who's calling them when they receive a call.
Assignee: nobody → standard8
User Story: (updated)
Assignee: standard8 → rgauthier
Attachment #8396416 - Flags: review?(standard8)
Comment on attachment 8396416 [details] [review]
Send an caller identifier when obtaining a call-url

Commented on PR, I think this needs a bit more work before we can land it.
Attachment #8396416 - Flags: review?(standard8) → review-
Comment on attachment 8396416 [details] [review]
Send an caller identifier when obtaining a call-url

Updated patch looks good. r=Standard8 with nits addressed.
Attachment #8396416 - Flags: review- → review+
As we wanted to get this landed and the nits were small, I've addressed them for you, and landed them here:

I've also synced the shared part of the patch with loop-client as we're looking to centralise that directory:

Leaving open until this actually lands on m-c.
Whiteboard: [fixed in gecko-dev]
Priority: -- → P1
Whiteboard: [fixed in gecko-dev] → [fixed in loop-ui-initial]
Depends on: 976109
Whiteboard: [fixed in loop-ui-initial] → [landed on loop-ui-initial]
backlog: --- → mlp+
Marking fixed as this has landed on loop-ui-initial and we believe the work here is done; loop-ui-initial will be cherry-picked to master at an appropriate time.
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [landed on loop-ui-initial] → [landed on loop-ui-initial][s=mlpnightly1, p=.25]
Target Milestone: --- → mozilla33
Please use meaningful string ids in the future.
Looks like this landed with tests. Does this need QA testing?
QA Contact:
Whiteboard: [landed on loop-ui-initial][s=mlpnightly1, p=.25] → [landed on loop-ui-initial][s=mlpnightly1, p=.25][qa?]
Flags: qe-verify-
Whiteboard: [landed on loop-ui-initial][s=mlpnightly1, p=.25][qa?] → [landed on loop-ui-initial][s=mlpnightly1, p=.25]
You need to log in before you can comment on or make changes to this bug.