Closed
Bug 1132336
Opened 10 years ago
Closed 10 years ago
[Calendar] notifications aren't closed during click if it was triggered while the calendar app was closed
Categories
(Firefox OS Graveyard :: Gaia::Calendar, defect)
Tracking
(blocking-b2g:2.2+, b2g-v2.2 verified, b2g-master verified)
People
(Reporter: mmedeiros, Assigned: mmedeiros)
References
Details
(Keywords: regression)
Attachments
(1 file)
46 bytes,
text/x-github-pull-request
|
gaye
:
review+
bajaj
:
approval-gaia-v2.2+
|
Details | Review |
steps to reproduce: 1. create a new event with a reminder; 2. close the calendar app before the notification is fired; 3. tap the notification (can be on the lockscreen, tray or toast); expected: it should open the calendar app on the correct event and delete the notification. actual: notification remains on the tray.
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → mmedeiros
blocking-b2g: --- → 2.2?
status-b2g-v2.2:
--- → affected
status-b2g-master:
--- → affected
Keywords: regression
Comment 1•10 years ago
|
||
Assignee | ||
Comment 2•10 years ago
|
||
Comment on attachment 8563716 [details] [review] [gaia] millermedeiros:1132336-notification-click-close > mozilla-b2g:master added a "tag" to all the notifications ("tag" works like an ID) so if the app creates multiple notifications for same URL it will consider as a single notification (will override the previous one). It also uses the "tag" to find and close all the notifications during `launch`, that way we use the same logic if app is open or closed avoiding this kind of error. also added some safeguards against the unlikely case of a busytime or event being null for some reason when the alarm is triggered (I think I saw it happen but was not able to reproduce).
Attachment #8563716 -
Flags: review?(gaye)
Assignee | ||
Updated•10 years ago
|
Target Milestone: --- → 2.2 S6 (20feb)
Comment 3•10 years ago
|
||
Comment on attachment 8563716 [details] [review] [gaia] millermedeiros:1132336-notification-click-close > mozilla-b2g:master LGTM!
Attachment #8563716 -
Flags: review?(gaye) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Updated•10 years ago
|
Keywords: checkin-needed
Comment 5•10 years ago
|
||
Pull request has landed in master: https://github.com/mozilla-b2g/gaia/commit/8df64a303797de9b678967d929d7a9215486eb8d
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•10 years ago
|
Assignee | ||
Comment 6•10 years ago
|
||
Comment on attachment 8563716 [details] [review] [gaia] millermedeiros:1132336-notification-click-close > mozilla-b2g:master [Approval Request Comment] [Bug caused by] (feature/regressing bug #): 1042939 [User impact] if declined: user will need to delete the notification manually and might cause confusion (it's a regression and bad UX) [Testing completed]: manual [Risk to taking this patch] (and alternatives if risky): very low risk, changes are local and should not cause conflicts [String changes made]: none
Attachment #8563716 -
Flags: approval-gaia-v2.2?
Updated•10 years ago
|
Attachment #8563716 -
Flags: approval-gaia-v2.2? → approval-gaia-v2.2+
Comment 7•10 years ago
|
||
v2.2: https://github.com/mozilla-b2g/gaia/commit/817f47167ffde2c6afb54fdb3fafbbca21296b07
Target Milestone: 2.2 S6 (20feb) → 2.2 S7 (6mar)
Comment 8•10 years ago
|
||
This issue is verified fixed on the latest Nightly 3.0 and 2.2 builds. Actual results: The notifications are dismissed when pressed on for calendar events even if the calendar is closed. Environmental Variables: Device: Flame 3.0 BuildID: 20150226010233 Gaia: 7894b929f1b0394f3c997f72a6482bc7813e758d Gecko: dd6353d61993 Gonk: e7c90613521145db090dd24147afd5ceb5703190 Version: 39.0a1 (3.0) Firmware Version: v18D-1 User Agent: Mozilla/5.0 (Mobile; rv:39.0) Gecko/39.0 Firefox/39.0 Environmental Variables: Device: Flame 2.2 BuildID: 20150226002503 Gaia: bf24aa57fa7760260ab05d1f53242c8d8ae59e83 Gecko: 363123044e61 Gonk: e7c90613521145db090dd24147afd5ceb5703190 Version: 37.0 (2.2) Firmware Version: v18D-1 User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
Status: RESOLVED → VERIFIED
Flags: needinfo?(ktucker)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
You need to log in
before you can comment on or make changes to this bug.
Description
•