Opened on 01/09/2018 at 10:40:31 AM
Closed on 07/12/2018 at 09:40:26 AM
Last modified on 08/06/2018 at 04:29:09 PM
#6238 closed change (fixed)
Download/store notifications.json
Reported by: | mario | Assignee: | |
---|---|---|---|
Priority: | P2 | Milestone: | Adblock-Plus-for-Samsung-Browser-1.1.6 |
Module: | Adblock-Plus-for-Samsung-Browser | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | #3725 | Platform: | Samsung Browser |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | yes |
Review URL(s): |
Description
Background
As preparation for #3725 and as an option for us to properly assess volume of use, we'd like to adopt the functionality to fetch and store the notifications.json before #3725 – which is more complicated and time consuming – is implemented.
What to change
Regularly fetch and store notifications.
Technical implementation TBD.
References
Notifications have been implemented in other extensions already. The changes can be found here:
Attachments (0)
Change History (6)
comment:1 Changed on 04/26/2018 at 12:49:10 PM by jwangenheim
- Milestone set to Adblock-Plus-for-Samsung-Browser-next
- Priority changed from Unknown to P2
- Ready set
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:2 Changed on 05/02/2018 at 08:19:33 AM by mario
- Milestone changed from Adblock-Plus-for-Samsung-Browser-1.1.5 to Adblock-Browser-for-Android-next
comment:3 Changed on 05/14/2018 at 10:22:08 PM by diegocarloslima
- Milestone changed from Adblock-Browser-for-Android-next to Adblock-Plus-for-Samsung-Browser-next
comment:4 Changed on 07/12/2018 at 09:39:58 AM by abpbot
comment:5 Changed on 07/12/2018 at 09:40:26 AM by jwangenheim
- Resolution set to fixed
- Status changed from reviewing to closed
comment:6 Changed on 08/06/2018 at 04:29:09 PM by traynard
- Verified working set
Note: See
TracTickets for help on using
tickets.
A commit referencing this issue has landed:
Issue 6238 - Download/store notifications.json