Closed
Bug 892528
Opened 11 years ago
Closed 11 years ago
[User Story] Closing a Notification
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Tracking
(blocking-b2g:koi+)
VERIFIED
FIXED
blocking-b2g | koi+ |
People
(Reporter: pdol, Assigned: qdot)
References
Details
(Keywords: feature, Whiteboard: [ucid:System5, FT:systems-fe, KOI:P1][systemsfe])
Attachments
(1 file)
46 bytes,
patch
|
jlal
:
review+
|
Details | Diff | Splinter Review |
User Story:
As a developer I want the ability to close notifications even when the originating app is not open so that I can ensure that notifications, which are no longer relevant are removed.
Acceptance Criteria:
1. If my app issues a close command for a currently active (shown in utility tray) notification, the notification is dismissed and removed from view.
2. If my app issues a close command for a notification that has already been dismissed (no longer shown in utility tray), nothing happens.
Assumption: W3C spec is adhered to
Updated•11 years ago
|
Blocks: koi-system-fe
Reporter | ||
Comment 1•11 years ago
|
||
After further discussion with engineering, the user story has been updated to only cover the scenario where the app is open (it is assumed that the app will open via system timers or an equivalent mechanism):
As a developer I want the ability to close notifications so that I can ensure that notifications, which are no longer relevant are removed.
Updated•11 years ago
|
Whiteboard: [ucid:System5] → [ucid:System5, FT:systems-fe, KOI:P1]
Comment 2•11 years ago
|
||
Note that there is one Gaia patch in bug 782211 that has not landed yet, I'll do it later today.
So (maybe) this bug will be fixed after this.
Comment 3•11 years ago
|
||
Just filed bug 899574 so that we can get the active notifications for an app, which is necessary to survive app restarts.
Comment 4•11 years ago
|
||
Added a bunch of bugs I filed as part of my tests in bug 855165. Some of them have simple patches if you feel like reviewing them.
Comment 5•11 years ago
|
||
Candice Serran changed story state to started in Pivotal Tracker
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → kyle
Assignee | ||
Comment 6•11 years ago
|
||
Once bug 903547 lands, this should work for any app using the new notification API, since they can just call close() on the notification and the code is already in to check for tags and close accordingly.
Updated•11 years ago
|
Flags: in-moztrap?(jsmith)
Comment 7•11 years ago
|
||
Maybe in this bug you can add some code in the UITest test app to be able to close a notification ?
The code is in /test_apps/uitest/js/newnotification.js and /test_apps/uitest/js/newnotification.html (I added these files in bug 901856)
Comment 8•11 years ago
|
||
(In reply to Julien Wajsberg [:julienw] from comment #7)
> Maybe in this bug you can add some code in the UITest test app to be able to
> close a notification ?
>
> The code is in /test_apps/uitest/js/newnotification.js and
> /test_apps/uitest/js/newnotification.html (I added these files in bug 901856)
julienw, I've done it in bug https://bugzilla.mozilla.org/show_bug.cgi?id=907375
It worked well here.
Assignee | ||
Comment 9•11 years ago
|
||
Attachment #794966 -
Flags: review?(jlal)
Comment 10•11 years ago
|
||
Comment on attachment 794966 [details] [diff] [review]
Patch 1 (v1) - Integration Tests for Closing Notifications
See github (some minor changes requested)
Attachment #794966 -
Flags: review?(jlal) → review-
Comment 11•11 years ago
|
||
Comment on attachment 794966 [details] [diff] [review]
Patch 1 (v1) - Integration Tests for Closing Notifications
https://github.com/qdot/gaia/commit/64339f4bfd563ca0ecaf0698e66db979c8df6898
Attachment #794966 -
Flags: review- → review+
Comment 12•11 years ago
|
||
wrong commit above...
in master: https://github.com/mozilla-b2g/gaia/commit/a3b860b817d4877c10b43c34efdc1d43695fd9f7
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 13•11 years ago
|
||
So I landed this but you still need a test for .onclose probably see https://bugzilla.mozilla.org/show_bug.cgi?id=908978
Flags: needinfo?(kyle)
Comment 14•11 years ago
|
||
Kyle Machulis changed story state to accepted in Pivotal Tracker
Comment 15•11 years ago
|
||
Kyle Machulis changed story state to accepted in Pivotal Tracker
Comment 16•11 years ago
|
||
Kyle Machulis changed story state to accepted in Pivotal Tracker
Updated•11 years ago
|
Blocks: b2g-notifications
Comment 17•11 years ago
|
||
Flags: in-moztrap?(jsmith) → in-moztrap+
Updated•11 years ago
|
Whiteboard: [ucid:System5, FT:systems-fe, KOI:P1] → [ucid:System5, FT:systems-fe, KOI:P1][systemsfe]
Updated•11 years ago
|
blocking-b2g: koi? → koi+
Assignee | ||
Updated•11 years ago
|
Flags: needinfo?(kyle)
Comment 18•11 years ago
|
||
Verified per completion of initial test pass.
Status: RESOLVED → VERIFIED
Keywords: verifyme
You need to log in
before you can comment on or make changes to this bug.
Description
•