Closed
Bug 1466676
Opened 7 years ago
Closed 7 years ago
Convert lodash .find() to native ES6
Categories
(Tree Management :: Treeherder: Frontend, defect, P3)
Tree Management
Treeherder: Frontend
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jonarea, Assigned: jonarea)
References
Details
Attachments
(1 file)
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/66.0.3359.181 Safari/537.36
Steps to reproduce:
Converting all _.find() functions to native .find() as outlined in meta Bug 1466494
Updated•7 years ago
|
Assignee: nobody → jonarea
Blocks: 1466494
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Priority: -- → P3
Comment 1•7 years ago
|
||
Updated•7 years ago
|
Attachment #8983905 -
Flags: review?(cdawson)
Comment 2•7 years ago
|
||
Requested changes in PR.
Comment 3•7 years ago
|
||
Comment on attachment 8983905 [details] [review]
Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3623
Requested an update. Clearing review for now.
Flags: needinfo?(cdawson)
Attachment #8983905 -
Flags: review?(cdawson)
Comment 4•7 years ago
|
||
Clearing my own need-info while I wait for fixes from the contributor.
Flags: needinfo?(cdawson)
Updated•7 years ago
|
Flags: needinfo?(cdawson)
Comment 5•7 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/9b3ba7149b865bc58be7eed284ae98be77a25d8f
Bug 1466676 - Refactor all lodash .finds to ES6 (#3623)
Comment 6•7 years ago
|
||
Comment on attachment 8983905 [details] [review]
Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3623
Great work! Thanks!
Flags: needinfo?(cdawson)
Attachment #8983905 -
Flags: review+
Updated•7 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•