Closed Bug 1184919 Opened 9 years ago Closed 7 years ago

Cleanup the thCloneJobs directive (clonejobs.js)

Categories

(Tree Management :: Treeherder: Frontend, defect, P3)

Product:
Tree Management
Component:
Treeherder: Frontend
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED DUPLICATE of bug 1409103

People

(Reporter: mdoglio, Unassigned)

Details

Clonejobs.js is a >700 lines file containing the logic to create the platform rows and their jobs. The code is not very readable and hard to maintain. We should invest some time refactoring it.
Priority: -- → P4
Component: Treeherder → Treeherder: Frontend
(In reply to Mauro Doglio [:mdoglio] from comment #0) > Clonejobs.js is a >700 lines file 1050 lines now..
Priority: P4 → P3
Summary: Cleanup the clonejobs directive → Cleanup the thCloneJobs directive (clonejobs.js)
I still have the branch from Casey converting clonejobs to reactjs. I'll resurrect that branch at some point in the near future as we begin our transition over. So that'll basically kill this file completely.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.