Closed Bug 1274172 (akita-multiuser-chat) Opened 8 years ago Closed 8 years ago

make chat support multiple users

Categories

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

Product:
Hello (Loop)
Component:
Client
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: Mardak, Assigned: dmosedale)

References

Details

(Whiteboard: [akita-alpha])

User Story

As a Hello developer and room user, I can chat with multiple users, so that we can more easily eat our own dogfood.

Acceptance criteria:
 * names for each user are displayed in the speech bubbles in the UI.
 * a random, stable display name is generated for each user
 * own messages received in the update stream are not displayed a second time

Tech checklist:
 * make sure the ParticipantStore and DataDriver propagate display name data and use
 * make views use state from the ParticipantStore to render display names correctly

Attachments

(1 file)

Link to Github pull-request: https://github.com/mozilla/loop/pull/480
40 bytes, text/x-github-pull-request
Mardak
: review+
Details | Review 
This could potentially be used to avoid the local echo because now we can realize a chat message was actually from ourself. We'll probably need to reference participant data from bug 1274168.
Rank: 9
Priority: -- → P1
Whiteboard: [akita] → [akita-alpha]
Assignee: nobody → dcritchley
Assignee: dcritchley → dmose
No longer blocks: akita-chat-displaynames
Blocks: akita-dogfood
User Story: (updated)
Depends on: akita-chat-displaynames
Summary: Include participant/sender information in chat messages → make chat support multiple users
Alias: akita-multiuser-chat
This is gonna need a few spinoffs to complete the work, but this is the minimal step forward.
Attachment #8761014 - Flags: review?(edilee)
Comment on attachment 8761014 [details] [review]
Link to Github pull-request: https://github.com/mozilla/loop/pull/480

r=Mardak with filing bugs and referencing them in the "XXX akita bug ##" comments and restoring some SENT tests with it.skips instead.
Attachment #8761014 - Flags: review?(edilee) → review+
Blocks: 1279042
Blocks: 1279054
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: