Opened 5 years ago

Closed 4 years ago

Last modified 4 years ago

#2274 closed change (fixed)

[Send notifications to a subset of the user base] Move notification parsing into a module

Reported by: fhd Assignee: fhd
Priority: P2 Milestone:
Module: Sitescripts Keywords:
Cc: Blocked By:
Blocking: #2162, #2275 Platform: Unknown
Ready: yes Confidential: no
Tester: Ross Verified working: yes
Review URL(s):

http://codereview.adblockplus.org/6308119894294528/

Description

Background

For #2162, we need to parse notifications in a handler. Currently this is happening in a management script, so we need to move it into a module. We need to keep the management script around for some time so we can roll out #2162 gradually.

What to change

Move notification parsing from generateNotifications.py to a new module.

Change History (6)

comment:1 Changed 5 years ago by fhd

  • Blocking 2275 added

comment:2 Changed 5 years ago by fhd

  • Summary changed from Move notification parsing into a module to [Send notifications to a subset of the user base] Move notification parsing into a module

comment:3 Changed 5 years ago by fhd

  • Owner set to fhd

comment:4 Changed 5 years ago by fhd

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:5 Changed 4 years ago by fhd

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:6 Changed 4 years ago by Ross

  • Tester set to Ross
  • Verified working set

This has been implemented/done.

Note: See TracTickets for help on using tickets.