Opened on 11/28/2017 at 03:31:26 PM

Closed on 12/11/2017 at 03:30:03 PM

#6104 closed defect (fixed)

Minified code in help center repository

Reported by: juliandoucette Assignee: ire
Priority: P3 Milestone: help.eyeo.com 1.0.0
Module: Websites Keywords:
Cc: ire, wspee Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29624561

Description (last modified by juliandoucette)

Environment

  • help.eyeo.com

How to reproduce

  • See /static/js/vendor

Observed behaviour

  • See minified code

Expected behaviour

  • Code should be minified and source mapped via build process
  • Uncompressed versions should be available in production e.g. "main.min.css" and "main.css"
  • Dependencies should be managed by npm and excluded from source code repository
  • Make license information machine readable (https://www.gnu.org/software/librejs/free-your-javascript.html)

Let's not fall into The JavaScript Trap.

Attachments (0)

Change History (5)

comment:1 Changed on 11/28/2017 at 03:35:21 PM by juliandoucette

  • Description modified (diff)

comment:2 Changed on 11/29/2017 at 11:36:25 AM by ire

  • Owner set to ire
  • Ready set

comment:3 Changed on 11/29/2017 at 02:30:08 PM by ire

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

comment:4 Changed on 12/11/2017 at 03:29:10 PM by abpbot

A commit referencing this issue has landed:
Issue 6104 - Minified code in help center repository

comment:5 Changed on 12/11/2017 at 03:30:03 PM by ire

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

Add Comment

Modify Ticket

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