Opened 5 years ago

Closed 4 years ago

#1284 closed change (fixed)

Remove id from AppInfo

Reported by: fhd Assignee: Mailkov
Priority: P3 Milestone:
Module: Libadblockplus Keywords:
Cc: Blocked By:
Blocking: Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

https://github.com/adblockplus/libadblockplus/pull/2

Description

Background

We don't need AppInfo::id. In Firefox, this field is used for the addon ID, on other platforms it is irrelevant.

What to change

Remove the id member variable from AppInfo.

Change History (6)

comment:1 Changed 4 years ago by Mailkov

Hi, I would like to work on this!!

comment:3 Changed 4 years ago by Mailkov

Can you assign it to me? So I will receive e-mail on this issue.

comment:4 Changed 4 years ago by fhd

  • Owner set to Mailkov

comment:5 Changed 4 years ago by fhd

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

comment:6 Changed 4 years ago by fhd

  • Resolution set to fixed
  • Status changed from reviewing to closed
Note: See TracTickets for help on using tickets.