Applicable to the standalone & desktop - if no-one has entered the room, we shouldn't display the feedback view. If someone else has entered the room, then we should give the user the option for feedback, even if they other party left first.

Comment on attachment 8538546 [details] [diff] [review] Bypass the feedback form if no-one has entered the room yet Review of attachment 8538546 [details] [diff] [review]: ----------------------------------------------------------------- ::: browser/components/loop/content/shared/js/activeRoomStore.js @@ +95,5 @@ > + // Tracks if the room has been used during this > + // session. 'Used' means at least one call has been placed > + // with it. Entering and leaving the room without seeing > + // anyone is not considered as 'used' > + used: false Note: we need to make sure we reset this value after the feedback has been given (so that re-entering the room without reloading the page works correctly as well).

I still want to get this fixed ASAP, but we'll likely only uplift it to Fx36 at this point.

(In reply to Mark Banner (:standard8) from comment #2) > Comment on attachment 8538546 [details] [diff] [review] > Bypass the feedback form if no-one has entered the room yet > > Review of attachment 8538546 [details] [diff] [review]: > ----------------------------------------------------------------- > > ::: browser/components/loop/content/shared/js/activeRoomStore.js > @@ +95,5 @@ > > + // Tracks if the room has been used during this > > + // session. 'Used' means at least one call has been placed > > + // with it. Entering and leaving the room without seeing > > + // anyone is not considered as 'used' > > + used: false > > Note: we need to make sure we reset this value after the feedback has been > given (so that re-entering the room without reloading the page works > correctly as well). The method feedbackComplete resets the state, so we should be fine here.

Comment on attachment 8540146 [details] [diff] [review] Bypass the feedback form if no-one has entered the room yet Review of attachment 8540146 [details] [diff] [review]: ----------------------------------------------------------------- Looks and works great!

Comment on attachment 8540146 [details] [diff] [review] Bypass the feedback form if no-one has entered the room yet Approval Request Comment [Feature/regressing bug #]: Rooms [User impact if declined]: User will be presented with a feedback form if they hit the "leave" button and no one had been in the room with them. Causes some user confusion and skew in the statistics. [Describe test coverage new/current, TBPL]: Includes test changes, for browser and standalone [Risks and why]: moderately low risk, very contained to loop; easy to test. Only asking for Aurora at this time and likely will live with it in 35. [String/UUID change made/needed]: none