Opened 5 years ago

Last modified 16 months ago

#1244 reviewing change

Cleanup of sitescript module dependencies

Reported by: matze Assignee: matze
Priority: Unknown Milestone:
Module: Unknown Keywords: reports sitescripts
Cc: fhd Blocked By:
Blocking: Platform: Unknown
Ready: no Confidential: no
Tester: Verified working: no
Review URL(s):

http://codereview.adblockplus.org/4805960266678272/

Description (last modified by matze)

Background

In the context of #240, we've already discovered a few unused import statements in the sitescripts.reports module. This ticket here shall serve as a context for cleanups done while migrating the reports.adblockplus.org service to a distinct server.

In http://codereview.adblockplus.org/4805960266678272/, the creation of this ticket was requested by @trev. Further patch-sets may follow, however.

Please also note that testing of this (minor) changes will be done in the context of #240. Thus, there shall be no changes in this context that are not related to this ticket.

Change History (6)

comment:1 Changed 5 years ago by matze

  • Description modified (diff)

comment:2 Changed 4 years ago by matze

  • Owner changed from matze to AAlvz

@AAlvz Please use your current setup for reports.adblockplus.org (see #240), apply the patch-set from this ticket's Review URL(s), and perform the same tests as done for the reports server - in order to verify that the changes do not affect the behavior in any way.

comment:3 Changed 4 years ago by AAlvz

LGTM, the reports can be send without issues and can be seen afterwards. Just as before

comment:4 Changed 4 years ago by AAlvz

  • Owner changed from AAlvz to matze

comment:5 Changed 4 years ago by matze

  • Status changed from new to reviewing

comment:6 Changed 16 months ago by fhd

  • Cc trev removed
Note: See TracTickets for help on using tickets.