Open Bug 1606717 Opened 4 years ago Updated 2 years ago

Discovery Stream placementsForEach maybe should be a reduce

Categories

(Firefox :: New Tab Page, enhancement, P2)

Product:
Firefox
Component:
New Tab Page
Type:
enhancement

Tracking

()

People

(Reporter: thecount, Unassigned)

Details

I think we can clean up a bit of the complexity in areas we use placementsForEach by instead using a reduce.

This is because in each case where we use placementsForEach we end up storing another array outside of the for each callbacks that we build inside the callbacks. I think a reduce would do this better.

Priority: -- → P2
No longer blocks: pocket-newtab
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.