[Loop] "Rooms list" screen is shown before message "Full room"

VERIFIED FIXED

Status

--
minor
VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: lolimartinezcr, Assigned: borjasalguero)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [Room1.1.1_TestRun2][In 1.1.1][loop approved for 1.1.1])

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
Fire E 2.0
Loop 1.1.1/d4d6d37

Pre-requisites
User A logged in Mobile Loop application with FxA
User B logged in Destkop Loop application.
User C logged in Mobile Loop application with MobileID.
User A has created and shared a room with contact B and C.

STRs:
1. User A has joined to room (owner).
2. User B has joined to room (invited).
3. User C taps on Message application, taps on message received, clicks in URL shared and clicks on "Join the conversation" in StandaloneUI.
4. User C selects camera and taps on "Join" button.

Actual result:
Room list is shown a few seconds and after appear message about "Full room". Please see video http://youtu.be/9my48JrtVD4

Expect
Message about "full room" is directly shown.
(Reporter)

Updated

4 years ago
Summary: [Loop] "Rooms list" is shown before message "Full room" → [Loop] "Rooms list" screen is shown before message "Full room"
Whiteboard: [Room1.1.1_TestRun2]
(Assignee)

Updated

4 years ago
Assignee: nobody → borja.bugzilla
(Assignee)

Comment 1

4 years ago
Created attachment 8556389 [details] [review]
Pull Request
Attachment #8556389 - Flags: review?(crdlc)
Comment on attachment 8556389 [details] [review]
Pull Request

Left a nit, LGTM, perfect!
Attachment #8556389 - Flags: review?(crdlc) → review+
(Assignee)

Updated

4 years ago
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Whiteboard: [Room1.1.1_TestRun2] → [Room1.1.1_TestRun2][Not in 1.1.1]
Although the Room list is still shown, the time you can see it has decreased a lot.
Minor issue but very low risk patch, so giving the approval for the uplifting.

Borja, could you be so kind of landing the patch in 1.1.1 branch? Thanks a lot!
Severity: normal → minor
Flags: needinfo?(borja.bugzilla)
Whiteboard: [Room1.1.1_TestRun2][Not in 1.1.1] → [Room1.1.1_TestRun2][Not in 1.1.1][loop approved for 1.1.1]
Whiteboard: [Room1.1.1_TestRun2][Not in 1.1.1][loop approved for 1.1.1] → [Room1.1.1_TestRun2][In 1.1.1][loop approved for 1.1.1]
(Reporter)

Comment 5

4 years ago
Tested and I can *still* see the rooms list but *inmediately* it changes to the message about busy.
Fire E 2.0
Loop 1.1.1/39a5284
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.