Meeting Minutes for March 17th, 2012

Reminders:

  • We have 3 more meetings before wrapping up, depending on the size/scope of your project, you may want to reserve the last week to polish and document your project
  • If you don’t think you’re going to be able to finish your projects, it’s critical that you inform one of the mentors
  • Status reports should posted at least one hour before the meeting, so please send them ASAP

Anthony

  • Framework should be able to handle additional mimetypes through extensions
  • Thumbnails will be rendered as HTML allowing the author to decide on the looks
  • Generate a new comment class for full-file review comments

Steven

  • Requested reviews on extension documentation
  • ReviewBot next top priority

Jim

  • Requested review on r2966
  • Will be knocking off bugs till the end of the semester or picking up one of Christian’s suggested projects

Yazan

  • Static media should be placed in “/reviewboard/static”, htdocs is off-limits now
  • Need to attach “–tracking-branch=origin/master” flag to post-review github branch
  • Use imgur.com to post screenshots, get an account to group images

Dave

  • Waiting on review request for r2918
  • Next project is possibly “better binary files”
  • Possibly select one of the projects suggested by Christian along with jimrrchen

Curtis

  • Having trouble placing “select_related()” within the code
  • “review_request = review_request.select_related(‘diffset_history__diffsets’)” is the gist of the idea,  Christian will provide code in the review

Wilson

  • Merge conflict occurred, instructed on how to resolve it
  • Some files may have changed locations and may need to re-apply changes
  • Mockup “new submitters page” using wireframe to a layout similar to the dashboard’s

Willer

  • Requirements for “weekly headlines project” are not clear, approach explained here: ” groups.google.com/group/reviewboard-dev/browse_frm/thread/7efca0ae1e6173fc”
  • Had trouble running unit tests, will contact dev mailing list for support

Christian’s suggested projects:

  • “if you select text in, say, the description box, it opens, which it shouldn’t do”
  • “if you have the box open, and yo udidn’t modify it, you’ll still be asked if you’re sure you want to leave the page, which is painful when trying to close a review request”
  • “when someone closes a review request, the update bubble that appears later is that the review request is “updated.” It should be smarter and say it was closed”
Advertisements

Leave a Reply

Fill in your details below or click an icon to log in:

WordPress.com Logo

You are commenting using your WordPress.com account. Log Out / Change )

Twitter picture

You are commenting using your Twitter account. Log Out / Change )

Facebook photo

You are commenting using your Facebook account. Log Out / Change )

Google+ photo

You are commenting using your Google+ account. Log Out / Change )

Connecting to %s