Closed Bug 1120897 Opened 11 years ago Closed 11 years ago

[Loop] Notification isn't shown name/surname about contact joins to room.

Categories

(Firefox OS Graveyard :: Gaia::Loop, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: lolimartinezcr, Assigned: crdlc)

References

Details

(Whiteboard: [Room1.1.1_TestRun1])

Attachments

(1 file)

Flame 2.0 Gecko-5f9dc54 Gaia-31d6c94 Loop version: master/51a04dd This bug is reproducible is this scenarios: -Scenario 1: User A logged in Mobile Loop application. User B logged in Desktop Loop application. -Scenario 2: User A logged in Mobile Loop application. User B logged in Mobile Loop application. Pre-requisites: User A logged in Loop application, has created and shared a room with User B. Scenario 1: User B saved in contact application with name, surname and email. Scenario 2: User B saved in contact application with name, surname and phone number. STRs: 1. Owner is already in the room. 2. User A keeps the App in Background. 3. User B joins the room shared by A. Actual result: Scenario 1: User A receives a notification saying "*Guest* has joined". Scenario 2: User A receives a notification sayint "*+346xxxxxxxx* has joined". Expected result: User A receives a notification saying "[Name/Surname] has joined".
Blocks: 1107894
Summary: [Loop] → [Loop] Notification isn't shown name/surname about contact joins to room.
QA Whiteboard: [TEF-Track]
Whiteboard: [Room1.1.1_TestRun1]
Assignee: nobody → crdlc
Status: NEW → ASSIGNED
Attached file Github pull request
Attachment #8549486 - Flags: review?(josea.olivera)
Comment on attachment 8549486 [details] [review] Github pull request LGTM. r=me, with nits addressed please. Thanks for taking care of it Cristian!
Attachment #8549486 - Flags: review?(josea.olivera) → review+
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Now the contact name/surname is shown in a notification received. Verified on Fire E 2.0 Loop 1.1.1/d4d6d37
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: