Author Archives: hellosjsu

Status Reports: April 19, 2014

Stephanie Su

1. What you accomplished this week

  • Worked on front end and back end to display widgets added by the user. This is now functioning as intended.
  • Incorporated suggestions from code review.

2. What you plan to do next week

  • Finish up back end for adding and removing widgets.
  • Fix some issues with removing widgets. When I remove multiple widgets and refresh the page, only the last removed widget stays removed. I need to figure out why my earlier post requests are being overridden and fix the bug.
  • Make the modal filter out widgets that have already been added. Right now, the widgets in the selection modal are hardcoded in; instead, I want to only display the ones that aren’t already shown on the dashboard.

3. What, if anything, is blocking you from making progress

  • None.

4. Any other questions

  • None.

Tami Forrester

1. What you accomplished this week

  • Fixed bug in reviews page to filter for reviews on private requests – now you can only see reviews if they are in public repos/groups, or you are in the repo/group
  • Addressed style comments
  • Took my review request out of WIP and began looking at the students projects list

2. What you plan to do next week

  • Start working on another project? I think either working on the Checklist Extension or adding new RBTools scripts would be interesting, not sure which one is most feasible in the time left.

3. What, if anything, is blocking you from making progress

  • None.

4. Any other questions

  • None.

Edwin Zhang

1. What you accomplished this week

  • I continued sandboxing, and took several out of WIP.
  • Sandboxing for NavigationBarHook and ReviewRequestApprovalHook were submitted
  • CommentDetailDisplayHooks and ActionHooks are out of WIP, but need to fix a couple things.

2. What you plan to do next week

  • Fix whatever I need to fix for my review requests
  • Look around for more things to sandbox, and if not, look for a new project.

3. What, if anything, is blocking you from making progress

  • I think I may be out of things to sandbox, I will take another look through the code base.

4. Any other questions

  • None, but if I can’t find anything to sandbox, I will get in contact with the mentors about a new project.

Tomi Aijo

1. What you accomplished this week

  • Finished r/5531 and will pull it out of WIP tomorrow
  • Cleaned up the code in r/5531
  • Changed bug infobox behavior as discussed in the last meeting; infobox does not show loading spinner but only appears if supported

2. What you plan to do next week

  • Start adding support for different bug trackers to features that r/5531 adds
  • Continue thinking about adding authentication to bug trackers

3. What, if anything, is blocking you from making progress

  • None

4. Any other questions

  • None

Iines Piesala

1. What you accomplished this week

  • I’ve been on easter holiday, but for tomorrow I’ll update the review request based on David’s comments and post different style pictures of the part 2 projects WIP.

2. What you plan to do next week

  • I plan to get more comments on my reviews, lock down the right style for highlighting the row (part 2) and get it out of WIP.

3. What, if anything, is blocking you from making progress

  • None

4. Any other questions

  • None

Joonas Lindholm

1. What you accomplished this week

  • I worked with the navigation menus for mobile: I included the search bar in the top menu and I tested how icons would look like on the second menu. I uploaded 3 new screenshots and the code here: https://reviews.reviewboard.org/r/5620/.

2. What you plan to do next week

  • Fix some functionality issues with the navigation bars and make the code better.

3. What, if anything, is blocking you from making progress

4. Any other questions

  • None.

Bhushan Ramnani

1. What you accomplished this week

  • Added support for svn repository for the beasnstalk post-receive hook [#5698]. Would appreciate some feedback. Added test cases for git repository. Started working on googlecode post-receive hook.

2. What you plan to do next week

  • Finish implementation and testing of googlecode post-receive hook.

3. What, if anything, is blocking you from making progress

  • I’m unable to perform setup-repo on the beanstalk svn repository. I get the error message: No Subversion repository found or selected for http://0.0.0.0:8080. .reviewboardrc not created.
  • I’ve added the svn repository through the admin and that is reflected in the api request http://0.0.0.0:8080/api/repositories/
  • The xml file contains the beanstalk-svn-hook-test repo (http://pastie.org/9094247). Unable to test the hook on svn repo.

4. Any other questions

  • None.

Mirai Akagawa

[Waiting on status report.]

Advertisements

Meeting Minutes: March 9, 2014

Announcements

  • If you guys have comments from us on reviews, and have fixed those comments, make sure to click Fixed on the issues (and make sure each one is actually Fixed, if you haven’t done so already)

Round-robin

Tami

  • The review request object has a starred attribute, though reviews don’t, and i was wondering if that is something you would want to add, or if the star would refer to the review request that the review was made for?
    • I think adding it for reviews would just complicate things. So, I don’t really see a point myself to being able to star reviews… it *might* be useful for a review requester to star reviews they need to address, but I think that’s just adding extra complexity.
  • Can I just scrap the star column then?
    • Yup.
  • For the frontend, should the reviews and review request tabs always be shown independent of the # of reviews someone has made?

    • I’d say yes. Better to be consistent.

Tomi

  • I have spent this week implementing the configuration UI. Things are already working besides dynamically adding new rules, but I know how to do that, just have not had time to do that.
    • Can you add a screenshot of it to your review request? Early screenshots are fine too 😀
  • There are still a number of open issues on my review request. I have not fixed them yet, those files are still under development and change frequently. I have been planning to first implement all features and then polish things.

Stephanie

  • I worked on fixing my code and started the front end for adding widgets. I saw that David reviewed my code, and I will get to that soon.

Joonas

  • I will post the code I used to make my screenshots so the mentors can take a look at my approach.
    • For screenshots on a retina display, I recommend using a tool like RetinaCapture. The high-dpi screenshots are very hard to review. Or at the very least, scaling down the screenshots by 50%. Actually I guess with this being Android, scaling down might be your only option. Can you post that code ASAP please?
  • I would like some comments on the design.
    • I think the design works.
  • I didn’t post the code so far because it was so bad and all over the place, but I will do it.
    • That’s perfectly fine, we still would like to see it, and we can comment on general things like the approach rather than code cleanliness.
    • Have you thought about the next step you’re going to take after this login form?
      • Not really.
  • The css now effects the registration form also so have to do testing with that also.
    • That might be something to investigate while you wait for feedback on this stuff. Now that you’ve got the media queries setup etc you might want to consider styling a few other things as well.

Mirai

  • [Excused – CMU on Spring Break]

Olessia

  • I’ve been working on the API for adding repositories. Testing for that is interesting… quite overwhelming. I don’t think I have questions I could ask now, I’m still exploring the code base, wrapping my head around things.

Anselina

  • I have a question about closing a review request: so review requests can be closed generally, without a user to close it?
    • View code can do anything. The issue here is that _find_review_request includes some permissions checks, so I think what we should do is split that up into two methods. One of those can just fetch the ReviewRequest object based on the params, and the other can use that and then add the checks.
    • It’d be nice to make that a decorator too.
    • Keep the views simple and let them take a parameter after fetching and after checks have been done assuming those would be useful to the new view.
  • Just to clarify, I should modify _find_review_request itself? Or create my own version of it where the two parts are split?
    • I think you should take the actual finding part out and move it into a new method. So modify it.
    • Leave _find_review_request as the one that does the permissions checks, and it can call your new thing. That way you don’t have to update tons of code.

Bhushan

  • [Excused – CMU on break]

Iines

  • I’ve made part 1 of my project, so the code line mumbers work now as a links, when you hover over those. Part 1 is making the links for the line numbers, and part 2 is to highlight the code line where the url is linked. I’ll post a picture of the code links after the meeting.
  • I will update the old review request.

Edwin

General tips

  • I’d like to request screenshots on the review requests if there’s anything to show.
  • I’d recommend going through the change and making sure the coding style matches the style we use throughout the rest of the codebase. Self-review is an important thing to practice. Go through the patch you have up, make any improvements you can think of, make sure all the styles (line lengths, var statements, braces, etc.) all match how we do things. Optimize the code best you can, and get it ready for us to try to give a final pass.
  • It’s generally a good idea to be on IRC whenever you’re working on this. If you get stuck ask questions often and early. For the most part, there is *usually* a mentor around. Early morning PST not included.