Opened 20 months ago

Last modified 9 months ago

#3063 reopened change

Back out fix for #3050, fix it in the conversion script

Reported by: fhd Assignee:
Priority: P1 Milestone:
Module: Adblock-Plus-for-iOS Keywords:
Cc: mario Blocked By:
Blocking: Platform: iOS
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description

Background

We have landed a hot fix for #3050 that modified easylist_with_acceptable_ads.json directly. While that has the same effect as far as google.com is concerned, it's a hack, and we really ought to fix this in the conversion script instead of having to make manual changes to the generated JSON.

What to change

Work around #3050 in the conversion script.

Change History (3)

comment:1 Changed 19 months ago by philll

  • Platform changed from Unknown / Cross platform to iOS

Added newly available platform value.

comment:2 Changed 18 months ago by fhd

  • Resolution set to duplicate
  • Status changed from new to closed

Considering that that #3050 was reopened and the change reverted, this issue is redundant now. We can just solve it properly in #3050.

comment:3 Changed 9 months ago by mario

  • Cc mario added
  • Resolution duplicate deleted
  • Status changed from closed to reopened

Reopened as #3050 was -- again -- fixed by applying a workaround (due to trying to fix this on short notice). Said workaround has been tested and works find and thus will stay in place until a proper fix was introduced.

Note: See TracTickets for help on using tickets.