[Loop] Contact's name is shown below the room name when owner cancels sending the url a contact to share the room

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
4 years ago
2 years ago

People

(Reporter: lolimartinezcr, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [Room1.1.1_TestRun1][sms-most-wanted][Room1.1.1_TestRun2])

(Reporter)

Description

4 years ago
Flame
2.0
Gecko-5f9dc54
Gaia-31d6c94
Loop version: master/51a04dd

Pre-requisites:
User A logged in Mobile Loop application.
User A has already created a room but has not invited anyone. The room name is shown in the room list.
User B is a contact saved  with name, phone number and email.

STRs:
1. User A: On Rooms list view, long tap on a room created by user A but not shared with anyone yet
2. Tap on Share room option
3. On Contact list view, select a valid contact (User B: with email and phone number)
4. Select share via phone number (SMS).
5. Once SMS app, tap on 'x' button to close the app (ER1)
5. Check Room list view (ER2)

Expected result:
ER1. The url is not shared with the contact. User A is taken back to Room list view
ER2. Contact's name is not shown below the room name

Actual result:
The url is not shared with the contact. User A is taken back to Room list view.  Contact's name *is shown* below the room name


Also is reproducible
STRs:
1. User A: On Rooms list view,  tap on a room created in ">"
2. One edit the rooom, tap on "Pick contact" button.
3. On Contact list view, select a valid contact (User B: with email and phone number)
4. Select share via phone number (SMS).
5. Once SMS app, tap on 'x' button to close the app 
5. Check Room list view
(Reporter)

Updated

4 years ago
Blocks: 1097745
(Reporter)

Updated

4 years ago
Whiteboard: [Room1.1.1_TestRun1]
I think that's the correct behavior because the Loop application is not able to know if the e-mail has been sent or not. 

So as soon as the e-mail application is opened and independently if later the e-mail has been sent, kept as a draft or discarded, it hould be updated with the contact shared in the invitees list and in the Room list.
(In reply to Maria Angeles Oteo (:oteo) from comment #1)
> I think that's the correct behavior because the Loop application is not able
> to know if the e-mail has been sent or not. 
> 
> So as soon as the e-mail application is opened and independently if later
> the e-mail has been sent, kept as a draft or discarded, it should be updated
> with the contact shared in the invitees list and in the Room list.

Sorry I wanted to say SMS application instead of E-mail application,according with the reported bug.

Anyway both applications should behave in the same way, and as soon as they are opened when selecting a contact to share a room the contact should be updated in the invitees list.

Checking this behavior with Loli, we have realized that E-mail is the one that is not working fine,so Loli, please could you open the corresponding bug for E-mail app. Thanks a lot!!!

I am closing this bug as invalid.
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → INVALID
(Reporter)

Comment 3

4 years ago
(In reply to Maria Angeles Oteo (:oteo) from comment #2)
> (In reply to Maria Angeles Oteo (:oteo) from comment #1)
> > I think that's the correct behavior because the Loop application is not able
> > to know if the e-mail has been sent or not. 
> > 
> > So as soon as the e-mail application is opened and independently if later
> > the e-mail has been sent, kept as a draft or discarded, it should be updated
> > with the contact shared in the invitees list and in the Room list.
> 
> Sorry I wanted to say SMS application instead of E-mail
> application,according with the reported bug.
> 
> Anyway both applications should behave in the same way, and as soon as they
> are opened when selecting a contact to share a room the contact should be
> updated in the invitees list.
> 
> Checking this behavior with Loli, we have realized that E-mail is the one
> that is not working fine,so Loli, please could you open the corresponding
> bug for E-mail app. Thanks a lot!!!
> 
> I am closing this bug as invalid.

New bug about Email created: https://bugzilla.mozilla.org/show_bug.cgi?id=1121452
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Component: Gaia::Loop → Gaia::SMS
Reopening the bug because this issue happens, but as far as I remember Julien told us that SMS is not going to change its behavior so I suspect that the bug will be solved as Won't fix or invalid
Currently there is no difference but maybe we could use postError instead of postResult when the user closes the activity without sending ? Or postResult with a result ?

Oleg, because you worked on SMS activity a lot, I think you could have a good opinion on this. What do you think?
Flags: needinfo?(azasypkin)
Also NI Andrew, to know if and how email handles this, because we should do the same thing.

Hey Andrew, do you return a special value or error to the activity caller when the user closes the activity before sending a mail?
Flags: needinfo?(bugmail)
We postError('cancelled') if the user backs out.  Versus our postResult('complete') if they actually send a message.
Flags: needinfo?(bugmail)
Thanks, we should definitely behave the same then.
Flags: needinfo?(azasypkin)
Whiteboard: [Room1.1.1_TestRun1] → [Room1.1.1_TestRun1][sms-most-wanted]
(Reporter)

Updated

4 years ago
Whiteboard: [Room1.1.1_TestRun1][sms-most-wanted] → [Room1.1.1_TestRun1][sms-most-wanted][Room1.1.1_TestRun2]
Duplicate of this bug: 1147171
Mass closing of Gaia::SMS bugs. End of an era :(
Status: REOPENED → RESOLVED
Last Resolved: 4 years ago2 years ago
Resolution: --- → WONTFIX
Mass closing of Gaia::SMS bugs. End of an era :(
You need to log in before you can comment on or make changes to this bug.