Opened 8 months ago

Closed 6 months ago

Last modified 3 months ago

#7027 closed change (fixed)

Adapt code to work with latest changes to FilterStorage.subscriptions

Reported by: greiner Assignee: greiner
Priority: P1 Milestone:
Module: User-Interface Keywords:
Cc: Blocked By:
Blocking: #7054 Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: yes
Review URL(s):

https://codereview.adblockplus.org/29907592/
https://gitlab.com/eyeo/adblockplus/abpui/adblockplusui/merge_requests/150
https://gitlab.com/eyeo/adblockplus/abpui/adblockpluschrome/merge_requests/10

Description (last modified by greiner)

Background

See https://gitlab.com/eyeo/adblockplus/abpui/adblockplusui/issues/219

What to change

See https://gitlab.com/eyeo/adblockplus/abpui/adblockplusui/issues/219

Hints for testers

  • Adding filters through Block element dialog should not fail.
  • Saving custom filters in desktop options page should add filters the user entered and remove filters the user deleted from the text area.
  • Filter lists should be correctly displayed in desktop options page, mobile options page and issue report data.
  • Disabled filters within filter should should be listed in the issue report using the "disabledFilters" property in the issue report data.

Change History (14)

comment:1 Changed 8 months ago by greiner

  • Owner set to greiner

comment:2 Changed 8 months ago by greiner

  • Description modified (diff)

Added hints for testers.

comment:3 Changed 8 months ago by greiner

  • Description modified (diff)

comment:4 Changed 7 months ago by jsonesen

  • Review URL(s) modified (diff)

comment:5 Changed 7 months ago by greiner

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

comment:6 Changed 7 months ago by greiner

  • Blocking 7054 added

comment:9 Changed 7 months ago by jsonesen

What do you think about reopening and using this ticket for the abpchrome repo as well? There is already a review: https://codereview.adblockplus.org/29907589/

Version 0, edited 7 months ago by jsonesen (next)

comment:11 Changed 7 months ago by greiner

  • Resolution fixed deleted
  • Status changed from closed to reopened

This change prematurely landed in the Mercurial master so I backed it out to allow for further fixes that need to be included in the ongoing 3.4 extension release. As soon as that one's done, we can reapply the changes.

comment:13 Changed 6 months ago by greiner

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

comment:14 Changed 3 months ago by ukacar

  • Verified working set
Note: See TracTickets for help on using tickets.