Opened 2 years ago

Closed 2 years ago

#5893 closed change (fixed)

Add application vendor information to issue reports

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

https://codereview.adblockplus.org/29587626/

Description (last modified by trev)

Background

Issue reports backend expects a vendor attribute on the <application> tag and will display "unknown" if it doesn't find one. Our current implementation doesn't add this attribute.

What to change

Populate the vendor attribute with window.navigator.vendor. While at it, upper-case application and platform name - the backend will display them as they are, and all lowercase doesn't look nice.

Change History (5)

comment:1 Changed 2 years ago by trev

  • Description modified (diff)

comment:2 Changed 2 years ago by trev

  • Owner set to trev

comment:3 Changed 2 years ago by trev

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

comment:5 Changed 2 years ago by trev

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