Summit

Code review comment for lp://staging/~cjohnston/summit/add-all-meetings

  1. add-all-meetings
  2. Merge into trunk
Revision history for this message
Joe Talbott (joetalbott) wrote :

Nothing jumps out at me. Those loops on L240 and L245 could get ugly if you have a lot of tracks and meetings but I don't thing that's the case right now.

I'm not familiar enough with the codebase to really comment on the templates and CSS stuff.

review: Approve

« Back to merge proposal