Closed Bug 840053 Opened 8 years ago Closed 7 years ago

[MMS][User Story] Message thread deletion

Categories

(Firefox OS Graveyard :: Gaia::SMS, defect, P1)

ARM
Gonk (Firefox OS)
defect

Tracking

(blocking-b2g:leo+)

RESOLVED WORKSFORME
1.1 CS (11may)
blocking-b2g leo+

People

(Reporter: pdol, Assigned: cassie)

References

Details

(Keywords: feature, Whiteboard: [LOE:M])

Attachments

(4 files)

UCID: Messages-062

User Story:
As a user, I want to be able to delete individual message threads.
The assumption is that this already works with the implementation of SMS, assigning to Bocoup to confirm.
Assignee: nobody → boaz
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)
Assignee: boaz → cassie
Whiteboard: [LOE:M]
Assignee: cassie → mike
The SMS application UI facilitates selecting individual message threads,

This behavior confirmed on Gaia@7a411f43b on a B2G device (Unagi) and FireFox Nightly.
The SMS application UI facilitates selecting individual message threads,

This behavior confirmed on Gaia@7a411f43b on a B2G device (Unagi) and FireFox Nightly.
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.
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.
(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)
(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)
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+ → -
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: b2g-mms-dom-api
moztrap #6700
Flags: in-moztrap? → in-moztrap+
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]
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
Closed: 7 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.