Closed Bug 1121866 Opened 9 years ago Closed 9 years ago

[Loop] No error message shown when trying to join to an expired room

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: isabelrios, Assigned: jaoo)

References

Details

(Whiteboard: [Room1.1.1_TestRun1][tef-triage])

Attachments

(1 obsolete file)

Flame 2.0
Gecko-5f9dc54
Gaia-31d6c94
Loop master version
User A logged in mobile loop app

STR
1. User A creates a room
2. Change the date of the device so the room appears as expired in Room details
3. Tap on the room name to join

EXPECTED
An error message should be shown: Oops... You can't join because the room has expired. Do you want to delete it?

ACTUAL
When user taps on the room the communication window is open, with the message that he is the only user in the room. I have waited for a while, but nothing else happens.
Assignee: nobody → josea.olivera
Status: NEW → ASSIGNED
See Also: → 1121996
See Also: → 1121924
Whiteboard: [Room1.1.1_TestRun1] → [Room1.1.1_TestRun1][tef-triage]
Comment on attachment 8549545 [details] [review]
Pointer to Github PR https://github.com/mozilla-b2g/firefoxos-loop-client/pull/454

Cristian, something like the code in the patch would do the trick. Would you mind to give us some feedback? Thanks!
Attachment #8549545 - Flags: feedback?(crdlc)
Comment on attachment 8549545 [details] [review]
Pointer to Github PR https://github.com/mozilla-b2g/firefoxos-loop-client/pull/454

LGTM, thanks a lot
Attachment #8549545 - Flags: feedback?(crdlc) → feedback+
Attachment #8549545 - Attachment is obsolete: true
The STR of this bug describes the correct behavior, as the Room is not really expired (it's still alive in the Server)and it's perfectly accessible so the actual Result is the correct one so closing the bug as Invalid.

What it was not correct, it's that in Room Details, the Room appears as Expired when it's not.

This issue has been fixed by Jose Antonio in bug 1121924
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: