Closed
Bug 892523
Opened 11 years ago
Closed 11 years ago
[User Story] Email Invocation from Notification
Categories
(Firefox OS Graveyard :: Gaia::E-Mail, defect)
Tracking
(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.
Updated•11 years ago
|
Component: Gaia::E-Mail → Gaia::Clock
Comment 1•11 years ago
|
||
A Pivotal Tracker story has been created for this Bug: http://www.pivotaltracker.com/story/show/53237775
Updated•11 years ago
|
Component: Gaia::Clock → Gaia::E-Mail
Updated•11 years ago
|
blocking-b2g: koi? → koi+
Updated•11 years ago
|
Blocks: koi-productivity
Comment 2•11 years ago
|
||
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.
Comment 3•11 years ago
|
||
There is a new app zip for this ticket:
https://bugzilla.mozilla.org/show_bug.cgi?id=892519#c14
Updated•11 years ago
|
Whiteboard: [ucid:Productivity2] → [ucid:Productivity2, FT:Productivity, KOI:P1]
Updated•11 years ago
|
Flags: in-moztrap+
Updated•11 years ago
|
Target Milestone: --- → 1.2 FC (16sep)
Comment 4•11 years ago
|
||
Pointer to Github pull-request
Comment 5•11 years ago
|
||
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)
Updated•11 years ago
|
Attachment #803750 -
Flags: review?(bugmail) → review+
Comment 6•11 years ago
|
||
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: 11 years ago
Resolution: --- → FIXED
Comment 7•11 years ago
|
||
That changeset also is the fix for bug 892522.
Comment 8•11 years ago
|
||
Created bug 935764 and bug 935772 to track the issues I found regarding this user story.
You need to log in
before you can comment on or make changes to this bug.
Description
•