Unnecessary code in populateTeamData?

[tbtommyb]

First of all, I like that you gave the holder object in this file a useful name and that the names of the methods map to the tables they're changing.

Can you explain the reason for using the len variable in populateTeams.team? I have seen another team do something similar and I don't see why you can't just put the final call to cb outside of the forEach loop. If it's because you need the id variable that's in scope of the callback then you could just assign it to an outer variable.

I like your general approach of building up the user teams, though I think that a combination of map and filter would be easier to read on line 71.