[MMS][User Story] Message thread deletion

RESOLVED WORKSFORME

Status

Firefox OS
Gaia::SMS
P1
normal
RESOLVED WORKSFORME
5 years ago
5 years ago

People

(Reporter: pdol, Assigned: cassie)

Tracking

({feature})

unspecified
1.1 CS (11may)
ARM
Gonk (Firefox OS)
feature
Dependency tree / graph
Bug Flags:
in-moztrap +

Firefox Tracking Flags

(blocking-b2g:leo+)

Details

(Whiteboard: [LOE:M])

Attachments

(4 attachments)

(Reporter)

Description

5 years ago
UCID: Messages-062

User Story:
As a user, I want to be able to delete individual message threads.
Depends on: 760065
Blocks: 842229
The assumption is that this already works with the implementation of SMS, assigning to Bocoup to confirm.
Assignee: nobody → boaz

Comment 2

5 years ago
In the current implementation you can delete entire message threads and individual messages within a thread.

Flagging Peter for NeedsInfo to confirm that the current implementation fulfills this requirement.
Flags: needinfo?(pdolanjski)

Updated

5 years ago
Assignee: boaz → cassie
(Assignee)

Updated

5 years ago
Whiteboard: [LOE:M]
(Assignee)

Updated

5 years ago
Assignee: cassie → mike
Created attachment 719512 [details]
Screen shot showing thread selection

The SMS application UI facilitates selecting individual message threads,

This behavior confirmed on Gaia@7a411f43b on a B2G device (Unagi) and FireFox Nightly.
Created attachment 719514 [details]
Screen shot showing selected thread

The SMS application UI facilitates selecting individual message threads,

This behavior confirmed on Gaia@7a411f43b on a B2G device (Unagi) and FireFox Nightly.
Created attachment 719516 [details]
Screen shot showing "delete thread" action confirmation

Once the user selects a message thread and indicates "Delete", the SMS application UI creates a modal dialog prompting for confirmation of message deletion.

This behavior confirmed on Gaia@7a411f43b on a B2G device (Unagi) and FireFox Nightly.
Created attachment 719518 [details]
Screen shot showing thread deletion progress

The SMS application UI indicates its state while a deletion operation is in progress.

This behavior confirmed on Gaia@7a411f43b on a B2G device (Unagi) and FireFox Nightly.
I've attached four screen shots to confirm that this behavior is implemented as specified by the bug description. This bug will remain open until such time as the MMS application is complete and confirmed to implement the user story.
(Reporter)

Comment 8

5 years ago
(In reply to Casey Yee [:cyee] from comment #2)
> In the current implementation you can delete entire message threads and
> individual messages within a thread.
> 
> Flagging Peter for NeedsInfo to confirm that the current implementation
> fulfills this requirement.

It does.
Flags: needinfo?(pdolanjski)
This functionality is already working. Could we close this type of bugs in order to be focus in the real work that is not implemented?
Flags: needinfo?(pdolanjski)
(Reporter)

Comment 10

5 years ago
(In reply to Borja Salguero [:borjasalguero] from comment #9)
> This functionality is already working. Could we close this type of bugs in
> order to be focus in the real work that is not implemented?

I'd prefer to verify that this works with MMS first (although it is the same app, this should be a quick verification once MMS is working).
Flags: needinfo?(pdolanjski)
Blocks: 849867
Per partner and release-driver discussions, marking blocking- until all MMS functionality in bug 849867 is complete, allowing it all to be uplifted at once to avoid SMS bustage.
blocking-b2g: leo+ → -

Updated

5 years ago
Flags: in-moztrap?
leo+ as this is a part of MMS and part of v1.1 to be included in leo+ queries. No_UPLIFT for now before the whole MMS is completed
blocking-b2g: - → leo+
Whiteboard: [LOE:M] → [LOE:M] [NO_UPLIFT]
Just a note, deleting threads that received (unviewable) MMS are unable to be deleted -- it just hangs on the "Deleting..." animation

Unagi, B2G 1.1.0.0-prerelease, built from SHA caaa81f7dfca6
Assignee: mike → cassie
Depends on bug 847736 instead of the meta bug. Bug 771458, accept array of message IDs, together with bug 854790, filtering with thread ID, may be handy here.
Depends on: 847736
No longer depends on: 760065

Comment 15

5 years ago
moztrap #6700
Flags: in-moztrap? → in-moztrap+

Updated

5 years ago
Assignee: cassie → gnarf37
As agreed in Madrid assigned to us https://wiki.mozilla.org/Gaia/SMS by now. I will talk with Steve Chung in order to check who will be in charge of this.
Assignee: gnarf37 → fbsc
Assignee: fbsc → cassie
Whiteboard: [LOE:M] [NO_UPLIFT] → [LOE:M]

Updated

5 years ago
Target Milestone: --- → 1.1 CS (11may)
Comment #10 is a noisy way of doing this. The MMS functionality is an iteration on the existing SMS code. Closing this, and QA will reopen when testing if for some reason the devs regress this functionality.

I tested this on nightly build, and WFM.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.