Closed Bug 1113705 Opened 10 years ago Closed 10 years ago

Joined Room should not show any icon in the Room list

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: oteo, Assigned: borjasalguero)

References

Details

Attachments

(1 file)

When someone sends me an invitation to join a Room (in this scenario nobody else was there but me) I am able to enter in the Room and when I leave that room, I can see in the Room list that the Room that I have just left is still occupied (blue Room icon in my Room list). But the room is really empty :(
Blocks: 1097738
Blocks: 1098534
Assignee: nobody → borja.bugzilla
Status: NEW → ASSIGNED
Renaming the title of this bug after checking with Pau and Monica that joined Rooms should never have an icon in the Room list as the invited user as soon as he left the room he's not aware of what happens in that Room. See WF in http://j.mp/loopwfs, Home Loop -> Rooms and Visual Rooms_log.v1.png in https://app.box.com/s/jt5v3rnvv3vqtyoj1n52/1/2717236524/24222713435/1 .
Summary: After joining and leaving an invited Room, the Joined Room in the Room list is still occupied → Joined Room should not show any icon the the Room list
Arnau is working also in bug 1113151 - Adapt Room list screen to the visual proposal, please be aware of it to decide if Arnau implement it on this bug or it's better to handle separately.
Flags: needinfo?(rnowmrch)
Flags: needinfo?(borja.bugzilla)
See Also: → 1113151
Attached file Pull Request
Flags: needinfo?(borja.bugzilla)
Summary: Joined Room should not show any icon the the Room list → Joined Room should not show any icon in the Room list
Attachment #8540160 - Flags: review?(crdlc)
Comment on attachment 8540160 [details] [review] Pull Request LGTM, landing because you are in PTO thought, thanks a lot
Attachment #8540160 - Flags: review?(crdlc) → review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Blocks: 1113151
Clearing ni as the patch is resolved ;)
Flags: needinfo?(rnowmrch)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: