Closed Bug 892523 Opened 7 years ago Closed 6 years ago

[User Story] Email Invocation from Notification

Categories

(Firefox OS Graveyard :: Gaia::E-Mail, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(blocking-b2g:koi+)

RESOLVED FIXED
1.2 FC (16sep)
blocking-b2g koi+

People

(Reporter: pdol, Unassigned)

References

Details

(Keywords: feature, Whiteboard: [ucid:Productivity2, FT:Productivity, KOI:P1])

Attachments

(1 file)

User Story:

As a user, I want to be taken directly from my notification to the specific email that I am being notified of when when the app is not open to save me time from finding the email app, launching it and finding the email. 


Acceptance Criteria:

1. After selecting a particular email notification from the utility tray, the email app is opened and that email body is displayed
2. If my selection from the utility tray includes a grouping of email notifications, the email app is opened and the email list for the mailbox that the group is related to is shown with the latest email in view
3. When selecting a particular email notification from the utility tray, the body of that email is loaded directly without first launching the email app, showing the mailbox list view and then displaying the email body.
4. When selecting a particular email notification group from the utility tray, the list view for that mailbox is loaded directly without first launching the email app, showing the default mailbox list view and then displaying the selected mailbox list view.
Component: Gaia::E-Mail → Gaia::Clock
A Pivotal Tracker story has been created for this Bug: http://www.pivotaltracker.com/story/show/53237775
Component: Gaia::Clock → Gaia::E-Mail
blocking-b2g: koi? → koi+
There is now a zip with the changes for this ticket, it can be found in bug 892519:
https://bugzilla.mozilla.org/attachment.cgi?id=794115

Right now that zip does the following:

* If only one new message is found during a sync, it creates a notification that when clicked goes to that specific message in the UI.
* If more than one message is found during a sync, then it creates a notification that when clicked goes to the message list for that email account.

Because of the current small space limitations for notifications, the multi-message notification will have this format:

    N New Emails
    Author 1, Author 2, Author 3

The notification normally cuts off the author list around Author 3.
There is a new app zip for this ticket:

https://bugzilla.mozilla.org/show_bug.cgi?id=892519#c14
Whiteboard: [ucid:Productivity2] → [ucid:Productivity2, FT:Productivity, KOI:P1]
Flags: in-moztrap+
Target Milestone: --- → 1.2 FC (16sep)
Comment on attachment 803750 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/12174

Switches over to using the new Notification approach now that [bug 910915](https://bugzilla.mozilla.org/show_bug.cgi?id=910915) has been fixed. Also needed to add an onclick handler since the message handler is not fired if the app is running. Once this lands, it will address the "email notification grouping" story too.

It also looks like our message handler is fired twice in some cases by the system, so a "uid" is added to the notification data so that we do not duplicate actions. I will file a bug for the system notification issue, but this allows us to land now, and may even be useful in the future.
Attachment #803750 - Flags: review?(bugmail)
Attachment #803750 - Flags: review?(bugmail) → review+
Fixed on master:
https://github.com/mozilla-b2g/gaia/commit/dd9e7936461d00b2b05f7cf59bdd227290aa9766

pull request:
https://github.com/mozilla-b2g/gaia/pull/12174
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
That changeset also is the fix for bug 892522.
Created bug 935764 and bug 935772 to track the issues I found regarding this user story.
Depends on: 935764, 935772
You need to log in before you can comment on or make changes to this bug.