1273512 - Show room's other participants presence when connected

Status: RESOLVED FIXED

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

Description

[Fernando Campo (:fcampo)]

As a user, I can see a representation of the room's other participants:
- https://mozilla.aha.io/attachments/6274825287365941975
- List of participants is shown next to 'Add Content' button, on it's left.
- User's own presence is not shown on user's side.
- Only online users are shown (presence tracked on Firebase)
- Participants are represented with the first letter of the username on a circle

Comment 1

[Fernando Campo (:fcampo)]

Assuming the presence is already tracked in Firebase, and showing it is only a matter of grabbing the username and transform it into a Letter Avatar

Comment 2

[GitHub Autolander Bot]

Attachment: [loop] mancas:bug1273512 > mozilla:akita

Comment 3

[Ed Lee :Mardak]

Comment on attachment 8758132 [details] [review]
[loop] mancas:bug1273512 > mozilla:akita

r=Mardak sorry for my accidental change of the API

Comment 4

[Manuel Casas Barrado [:mancas]]

Ed, as you can see in the tech list: 

- User's own presence is not shown on user's side.

So...how can we filter the owner? Is the userId stored in the room object?

Comment 5

[Pau Masiá [:Pau]]

Manu is correct. When being alone in a room, you don't need to see you are there because you already know it. You want to know when another participant joins, maybe just to be informed, maybe to interact with them...there may be many reasons. 

But in the end, the conclusion is that for the local user there's no avatar displayed and when guest joins, the avatar is displayed.

Comment 6

[Ed Lee :Mardak]

(In reply to Manuel Casas Barrado [:mancas] from comment #4)
> So...how can we filter the owner?
I think we can just assume `ParticipantStore` doesnt include the current user as part of its accessor method. This would be done via bug 1274172 when it adds the current participant. And if that bug doesn't, we can fix it in a followup.

Comment 7

[Ed Lee :Mardak]

https://github.com/mozilla/loop/commit/9589a0678eff6c47251152a4ad05bed2a3a66cbc

dmose pushed an extra commit to fix a `porps` typo:
https://github.com/mozilla/loop/commit/926459fe8cbcc712ce2c763986ac661b38efd2b9