Bug 1274172 (akita-multiuser-chat)

make chat support multiple users

RESOLVED FIXED

Status

Hello (Loop)
Client
P1
normal
Rank:
9
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: Mardak, Assigned: dmose)

Tracking

Firefox Tracking Flags

(Not tracked)

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 attachment)

(Reporter)

Description

2 years ago
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)

Updated

2 years ago
Blocks: 1275104
Assignee: nobody → dcritchley
Assignee: dcritchley → dmose
(Assignee)

Updated

2 years ago
No longer blocks: 1275104
(Assignee)

Updated

2 years ago
Blocks: 1271876
User Story: (updated)
Depends on: 1275104
Summary: Include participant/sender information in chat messages → make chat support multiple users
(Assignee)

Updated

2 years ago
Alias: akita-multiuser-chat
Blocks: 1278584
(Assignee)

Comment 1

2 years ago
Created attachment 8761014 [details] [review]
Link to Github pull-request: https://github.com/mozilla/loop/pull/480

This is gonna need a few spinoffs to complete the work, but this is the minimal step forward.
Attachment #8761014 - Flags: review?(edilee)
(Reporter)

Comment 2

2 years ago
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+
(Assignee)

Updated

2 years ago
Blocks: 1279042
(Assignee)

Updated

2 years ago
Blocks: 1279054
(Assignee)

Updated

2 years ago
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.