Opened on 03/17/2016 at 09:36:18 PM

Closed on 03/18/2016 at 06:03:45 PM

Last modified on 05/25/2016 at 02:10:43 PM

#3829 closed change (fixed)

Merge add-subscription and subscriptions.add messages

Reported by: sebastian Assignee: sebastian
Priority: P3 Milestone: Adblock-Plus-1.12-for-Chrome-Opera-Safari
Module: User-Interface Keywords:
Cc: greiner Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Ross Verified working: yes
Review URL(s):



Currently, there are two very similar message types add-subscription and subscriptions.add. The former is the only one in the adblockplusui repository that still uses the old naming scheme, and the only difference is that it brings up the options page asking the user to confirm adding the given subscription, while the later silently adds the given subscription.

What to change

Merge add-subscription message type into subscriptions.add and adapt the calling code.

Attachments (0)

Change History (7)

comment:1 Changed on 03/17/2016 at 09:44:26 PM by sebastian

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

comment:2 Changed on 03/18/2016 at 10:47:08 AM by greiner

  • Ready set

comment:3 Changed on 03/18/2016 at 10:47:26 AM by greiner

  • Priority changed from Unknown to P3

comment:4 Changed on 03/18/2016 at 01:31:55 PM by abpbot

A commit referencing this issue has landed:

comment:5 Changed on 03/18/2016 at 06:03:20 PM by abpbot

A commit referencing this issue has landed:

comment:6 Changed on 03/18/2016 at 06:03:45 PM by sebastian

  • Milestone set to Adblock-Plus-for-Chrome-Opera-Safari-next
  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:7 Changed on 05/25/2016 at 02:10:43 PM by Ross

  • Tester changed from Unknown to Ross
  • Verified working set

Adding subscriptions still works as expected after changes (except for subscribe link issue in older Chrome versions).

Chrome 31, 45, 50 / Windows 7
Chrome 32, 49 / OS X 10.11
Chrome 49 / Ubuntu 14.04

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from sebastian.
Note: See TracTickets for help on using tickets.