#223 Fixed the rendering problem by replacing forEach loop
Merged by dustymabe. Opened by gursewak.
Unknown source master

Download 223.patch

forEach is not promise-aware so it was creating a problem with rendering all the streams. We could only view four streams when we had hard coded to view five of them. Switched it with the classic for loop and added the promises array instead of returning within the loop (since for loop stops when we return something).

LGTM - my only suggestion would be to add more comments in the code about the subtlety here.

Looks sane to me.

rebased onto 63212aaed175c1176c035f5ff547921567bebf24

rebased onto 2f4719dca99835ac633aaf1e78edffc4226897a1

Pull-Request has been merged by dustymabe

Metadata