Closed Bug 1462625 Opened 6 years ago Closed 6 years ago

buildbot jobs don't load summaries anymore

Categories

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

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bhearsum, Assigned: emorley)

References

Details

Attachments

(1 file)

There's errors like 'TypeError: "({title:"Buildername"})' in the Browser Console.
Regression from:
https://github.com/mozilla/treeherder/commit/be3c6e8bc9caf31584196fec40d14d243adb4a39

Array.prototype.findIndex() only supports being passed a predicate function only:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/findIndex

...whereas _.findIndex() supported the _.matches() style shorthand:
https://lodash.com/docs#findIndex
Assignee: nobody → emorley
Blocks: 1413156
Status: NEW → ASSIGNED
Component: Treeherder → Treeherder: Frontend
Priority: -- → P1
Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/f8f260c165ad336bed4fcfc3325e572c5ab72c32
Bug 1462625 - Fix TypeError when selecting buildbot jobs (#3561)

This was changed from `_.findIndex()` to `Array.prototype.findIndex()`
in #3537, however was using the former's `_.matches()` shorthand
style, which isn't supported by the ES6 equivalent.

https://lodash.com/docs#findIndex
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/findIndex
This was deployed to production earlier.
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: