Archive

Archive for the ‘Status Reports’ Category

Status Reports – April 7, 2012

April 7, 2012 Leave a comment

Anthony

Status:

  • Tried to address issue 2551.
  • Updated the code of my existing review requests to reflect changes have been made since.

Next Steps:

  • Address conflict issues between r/2964 and r/2859 depending on which gets committed first.

Roadblocks:

Dave

Status:

  • Fixed a bug: update notification popping up when it shouldn’t. (r/3043)
  • Iterated capabilities for RBTools. (r/2918)
  • Looked into inline editor. (r/2991)

Next Steps:

  • Make sure projects are complete before end of term.

Roadblocks:
None

Steven

Status:

  • Restructured the Review Bot architecture, added features and a little more polish. (r/3042)
  • Updated Web Hooks extension. (r/2855)
  • Updated extension documentation. (r/2950)

Next Steps:

  • Hopefully have code committed.

Roadblocks:

  • Reviews.

Wilson

Status:

  • Looking for ship it status on r/2828

Next Steps:

  • Figure out data grids.

Roadblocks:

  • Having trouble understanding datagrids, hoping to get help.

Jim

Status:

  • Shipped r/3000 r/3001.
  • Need a ship-it on r/3033, unless there are further reviews.
  • Need someone to take a look at r/2966 and figure out what went wrong… I refactored the code but it is currently broken, not sure why.
  • Looking into a bug with typeahead reviewers, a bug that I opened in the beginning of term and never really get a chance to fix. (issue 2459)

Next Steps:

  • Hopefully finish r/2966.
  • Fix the bug that I discovered.

Roadblocks:

Yazan

Status:

  • Need review on r/2964.
  • Working on update issue status of deleted screenshot/file attachment bug.
  • Working on incremental expansion.

Next Steps:

  • Submit bug-fix/incremental-expansion patch.

Roadblocks:
None.

Wei Lu

Status:

  • Not too much done last week, finally finished other course projects.

Next Steps:

  • Concentrate working on Review Board.

Roadblocks:
None.

Categories: Status Reports

Status Reports for March 31st, 2012

March 31, 2012 Leave a comment

Yazan Medanat

Status:
Next Steps:
  • Finish incremental expansion of diffs (after block is resolved)
  • Update r2964 based on feedback
Road Blocks:

Dave Druska

Status:
  • Iterated reviews, need feedback on 2991 (look at my reply to Christian)
  • Sent out a bunch of reviews
Next Steps:
  • Finish my 2 things, tackle some bugs next
Road Blocks:
  • None

Steven MacLeod

Status:
Next Steps:
  • Finish extension documentation
  • Finish WebAPI updates
  • Try and get ReviewBot to a usable state (no more hard-coding etc.)
Road Blocks:

Jim

Status:
Next Steps:
  • Kepp building out the github-like diff rendering
Road Blocks:

Anthony Mok

Status:
  • Addressed the concerns in the review for the mimetype handler
Next Steps:
  • Work on feedback for the mimetype handler
  • Figure out how to only use handlers from enabled extensions.
Road Blocks:
  • None

Curtis Muller

Status:
  • N/A
Next Steps:
  • N/A
Road Blocks:
  • N/A

Willer

Status:
  • Fixed a few issue and thought more about the weekly headlines project
  • Being busy last week too
Next Steps:
  • Finish and close the two projects with whatever they cost
Road Blocks:
  • Have an presentation assignment this afternoon, so not sure if can attend the meeting on time or not.

Wilson Yeung

Status:
  • Using a new wireframe software (invisionapp.com )
Next Steps:
  • complete wireframe
  • finish 2282 bug
Road Blocks:
  • None
Categories: Status Reports

Status Reports – March 24, 2012

March 24, 2012 Leave a comment

Anthony

Status:
- I submitted an overview + implementation of the mimetype handling framework:  http://reviews.reviewboard.org/r/2967/.
Next Steps:
- Implement full-file reviews.
Roadblocks:
- I’m waiting on a response to the about review request.
- Discussion on how to integrate full-file reviews into the comment infrastructure.

Steven

Status:
- Updated WebHooks extension (http://reviews.reviewboard.org/r/2855/)
- Investigated and filed an issue about extension evolutions
 (http://code.google.com/p/reviewboard/issues/detail?id=2548)
- Discovered another modification to Review Board that ReviewBot requires, spoke with ChipX86 about re-scoping project.
- http://reviews.reviewboard.org/r/2957/ needs a push to master
Next steps

- Continue with ReviewBot

Roadblocks

- None

Jim

Status

- Need some reviews/ship-its on http://reviews.reviewboard.org/r/3000/ and http://reviews.reviewboard.org/r/3001/
- Also need some reviews on http://reviews.reviewboard.org/r/2966/, mostly just the added new method get_simple_chucks() in diffutil.py, the template side is still work in progress

Next steps

- Work on the github-like description change rendering
- Find other mini-projects/bugs

Roadblocks

- None

Dave

Status
-Waiting on review request http://reviews.reviewboard.org/r/2918/
-Submitted review request to improve the inline editor http://reviews.reviewboard.org/r/2991/
Next steps
-Find another mini project
-Possibly iterate capabilities upon receiving review
Roadblocks

-None

Wilson

Status

- Creating mockups for 829

Next steps

- more mockups for 829

Roadblocks

- Need to figure out where to place a snippet of code for r2828

Curtis

Status

-Implemented and tested Christian’s changes in r2810.
-Read through and commented on extension system documentation (r2950).

Next steps

-Get rating extension working.

Roadblocks

-Could use a quick review of http://reviews.reviewboard.org/r/2810/ to make sure Christian’s changes were applied correctly.

Yazan

Status
- Still working on incremental expansion
- Summary table moved with filters, stuck on new static CSS directory
Next steps

- Finish incremental expansion, post summary table

Roadblocks

- Stuck on new static CSS for 2 reviews

Wei

Status

- Worked on weekly headlines. Busy week, not too much done.

Next steps

- Continue. Work more.

Roadblocks

- None

Categories: Status Reports

Status Reports – March 17, 2012

March 17, 2012 Leave a comment

Anthony

Status
  • Started to add a framework to deal with file attachments according to their mimetype.
Next steps
  • Continue working on this framework.
  • Start work on the full-file review.
Roadblocks
  • Questions: Should administrators be able to extend this framework to handle additional mimetypes without modifying the reviewboard code?
  • What sort of thumbnails can be expected outside of image mimetypes?

Steven

Status
Next steps
  • Finish extension documentation
  • Finish up WebHooks now that extension upgrades are committed
Roadblocks
  • None

Jim

Status
  • Finally close to getting description diff rendering done. Will post the review once my vm is working
  • the inter-diffs are not show as it should. currently investigating and fixing CSS properies
  • Need reviews on my code.
Next steps
  • Finish my current project
Roadblocks
  • None

Yazan

Status
  • Waiting on feedback for r2881
  • Waiting on feedback for r2964 (Will post once roadblock is resolved)
  • Working on incremental expansion
Next steps
  • Make changes according to feedback
Roadblocks
  • htdocs/rb/static folder is ignored by git, not sure where to edit my CSS.

Dave

Status
  • Submitted review request for next version of capabilities
Next steps
  • Finish capabilities, start next project
Roadblocks
  • None

Curtis

Status
  • Worked on the backend for the rating extension
  • Feeling defeated regarding .select_related() on r2810
Next steps
  • Get the backend of the rating extension working.
  • Move on to front end after it is tested and working
Roadblocks
  • r2810 show_related(), I feel like I know how it should work, but I can’t get it working.

Wilson

Status
  • conceptually mapped solution for r2828
  • conceptually mapped solution for bug 829
    (conceptual because my git is failing)
Next steps
  • fix missed bug in r2828
  • start coding bug 829
Roadblocks
  • git

Wei

Status
  • Posted some starting code to r/2959 about Weekly Headlines
Next steps
  • Continue working on r2959
  • Finish testing r/2891
Roadblocks
  • Need specific requirements for Weekly Headlines
  • Had bugs when running unit test on fresh copy
Categories: Status Reports

Status Reports for March 10th, 2012

March 10, 2012 Leave a comment

Yazan Medanat

Status:
  • Updated environment
  • Working on summary table/incremental expansion
Next Steps:
  • post-review work
Road Blocks:
  • None

Dave Druska

Status:
  • Submitted patch for client-side capabilities
  • Submitted patches for bugs caused by the timezone merge
Next Steps:
  • Complete capabilities
Road Blocks:
  • None

Steven MacLeod

Status:
Next Steps:
  • Finish Documentation.
Road Blocks:
  • None.

Jim

Status:
  • Just finished a crazily-working, sleep-deprived week, finally some free time
  • Template tags are still not working for me. Really not sure why. Going to read the Django doc
Next Steps:
  • Continues working on description rendering
Road Blocks:
  • None

Anthony Mok

Status:
  • Unfortunately, I’ve been busy this week too, but that is going to end early next week.
Next Steps:
  • Get these school projects out of the way
  • Work on full-file review
Road Blocks:
  • None

Curtis Muller

Status:
  • Read about select_related(), and trying to implement it.
Next Steps:
  • Resolve the last issue on r2810
  • Continue working on rating-extension
Road Blocks:
  • Could use a comment on my comment on the open issue in r2810.
  • I had to come into work today, so I will do my best to be attentive during the meeting.

Willer

Status:
  • Followed up with comments on r2891; added “multiple draft owned by
  • separate users” support
Next Steps:
  • Unit test r2891
Road Blocks:
  • A small question about admin command

Wilson Yeung

Status:
  • Out of town this weekend
Next Steps:
  • N/A
Road Blocks:
  • N/A
Categories: Status Reports

Status Reports – Mar 3, 2012

March 3, 2012 Leave a comment

Anthony

Status:

  • Chose full-file review for next project.

Next Steps:

  • Work on full-file review when time permits.

Roadblocks:

  • Midterms and assignments.

Curtis

Status:

  • Posted changes for r/2810.
  • Made some progress on my extension.

Next Steps:

  • Spend more quality time with review-extension.

Roadblocks:

  • Need more hours in the day.

Dave

Status:

  • Researched all SCM diff generators for moved files.
  • Posted a review request for post-review with moved file capabilities.
  • Working on post-review “capabilities” feature which will work in unison with diff moved files. Christian completed the server side component (see RB commit 1fa7eb3).

Next Steps:

  • Finish client-side “capabilities”.

Roadblocks:

  • None.

Jim

Status:

  • Still struggling to get rendering working.

Next Steps:

  • Find time to work on it more, pretty busy week for me.

Roadblocks:

  • Possibly need Christian’s help.

Steven

Status:

  • Busy, Minimal progress.

Next Steps:

  • Close off extension system upgrades.
  • Update WebHooks and post a “final” review request.
  • Put ReviewBot up for review.

Roadblocks:

  • Fine for now.

Wei Lu

Status:

  • Researched “send email” and “django-admin command” for project “weekly headlines”.

Next Steps:

  • Ask for expertise and start coding.

Roadblocks:

  • For project “Tracking user changes” (r/2891), no comments since last meeting.

Wilson

Status:

  • Getting better.

Next Steps:

Roadblocks:

  • None.

Yazan

Status:

  • Busy week, not much was done.
  • Updated environment.

Next Steps:

  • Submit moved summary table for feedback.
  • Submit incremental expansion for review.

Roadblocks:

  • Lack of time this week.
Categories: Status Reports

Status Reports – Feb 25, 2012

February 25, 2012 Leave a comment

Anthony

Status:

  • Addressed database optimization issues with issue tracker integration.

Next:

  • Submit local changes to the extension generator for code review.
  • Choose another project.

Roadblocks:

  • Addressed database optimization issues with issue tracker integration.

Curtis

Status:

  • Posted work in progress for social extension. r2900
  • Quite sick this past week.

Next Steps:

  • Continue working on extension.
  • Fix last comment from Christian in r2810.

Roadblocks:

  • Anthony – What was the command/commands to install an extension to a reviewboard instance again? I have used it once and have since forgotten. I will add it to the extension docs once I know it again for next time.
  • Is there a way to use a template hook to call a view instead of just rendering a template?

Dave

Status:

  • Posted review request for the better moved files project
  • Fixed issues brought up in reviews

Next Steps:

  • Look into other SCMs

Roadblocks:

  • None

Jim

Status:

  • ran into vm problems (corrupted vmdk file), luckily I recovered most of my files
  • not sure about how to render the diff’s, hence unable to put up screenshots, really need Christian/Daivd’s help

Next step:

  • speed up things quickly since I am quite behind in my progress because of the vm problem

Roadblocks:

  • need Christian/Daivd’s help with diff rendering

Steven

Status:

Next Steps:

  • Close off extension system upgrades
  • Update WebHooks and post a “final” review request
  • Put ReviewBot up for review

Roadblocks:

  • Extension admin sites is blocked on django bug (If someone could triage, that would be great) / redesign of recent actions log
  • WebHooks extensions is blocked on extension upgrades

Wei Lu

Status:

  • finished most of “tracking user changes” project; updated r/2891

Next step:

  • follow up with more comments incoming
  • choose another project

Roadblocks:

  • currently, none

Wilson

Status:

  • recovering from hospital visit

Next Steps:

  • look for a new project
  • possibly looking for a “ship it” designation on previous project

Roadblocks:

  • help finding a new project

Yazan

Status:

  • Upgrading to Django 1.4b
  • Started working on incremental expansion
  • Waiting on reviews for r2881, r889

Next Steps:

  • Submit moved summary table for feedback (after Django update)
  • Submit incremental expansion for review

Roadblocks:

  • None
Categories: Status Reports Tags:

Status Reports – Feb 18, 2012

February 18, 2012 Leave a comment

Anthony

Status:

- Submitted Issue Tracking integration for code review

- Addressed very minor issue with SettingsForm-generated admin pages.

Next Steps:

- Address code reviews for Issue Tracking integration

- Look into adding options for SettingsForm to the extension generation script

Roadblocks:

- None

Curtis

Status:

- Added a Template Hooks section to the Building Extensions documentation.

- Made a couple revisions to Interdiffs.

- Started work on a prototype version of the reviewboard-social extension.

Next Steps:

- Continue working on the social extension.

Roadblocks:

- Currently switching over to a new mac, found out linux wont install on a MacBook Pro 8,2. Installing in a VM today.

- Yay new mac!

Dave

Status:

- Moved files works but doesn’t save chunks yet

- rbtools updated to pass -M flag

- Reading week! Will get lots of stuff done this week.

Next Steps:

- Get moved files to save chunks

- Detect moves on other version control systems in rb and rbtools

Roadblocks:

- Midterm week, not much free time but will make up for it this week.

Jim

Status:

- Made some progress with changed Description/Testing, have some basic functionality down

- Still stuck on the comment object not correctly sync’ed bug..will keep investigating

Next Steps:

- Working on building a prototype for the diff of changed Description/Testing section, should be able to come up with screenshots by end of weekend

Roadblocks:

- Busy week with interviews/midterms/assignments, will work more next week(reading week) to catch things up


Steven

Status:

- Made a small change to djblets (committed to master http://reviews.reviewboard.org/r/2886/)

- Added admin sites for extensions in djblets ( http://reviews.reviewboard.org/r/2888/ )

- Updated Web Hooks extension ( http://reviews.reviewboard.org/r/2855/ )

- Worked on ReviewBot.

Next Steps:

- Add the ability for extensions to have default settings.

- Fix issues with extension admin sites.

- Update Web Hooks to use new extension capabilities.

Roadblocks:

- Need more information on how extension admin sites should act / input on a bug

- Has anyone figured out how I should implement review bots access to the code (continuation from last week)

- Web Hooks depends on my extension changes

Wei Lu

Status:

- Could not make any progress in testing code due to django evolution

Next Steps:

- Get over the roadblock with assistance

Roadblocks:

- Stuck with django evolution for >10 hours; tried everything but it just didn’t work for me

Wilson

Status:

- made progress in understanding javascript, playing with code

Next Steps:

- look for new project

- making the comment box reflect changes made in the database

Roadblocks:

- figuring out relation between different coding languages

Yazan

Status:

- Required fields almost done

- Working on improved summary table

Next Steps:

- Submit improved summary table

- Pick up a new task

Roadblocks:

- Concern: Possible code conflict with r2859

Categories: Status Reports

Status Reports – Feb 11/12

February 11, 2012 Leave a comment

Anythony

Status

  • Picked up a new project (Issue Tracker Integration)
  • Got the web-UI aspects working

Next Steps

  • Do more testing and post a code review

Roadblocks

  • I am not sure what the best way is to test the e-mail notifications fetaure.

Curtis

Status

  • Implemented Christian’s and Mike’s changes, moved logic to the view. Could use a quick review (ship its hopefully).http://reviews.reviewboard.org/r/2810/
  • Used Antony’s script to generate my extension stub. Started working on the social extension backend, very basic functionality for the controller>model>view>dashboard_template working.

Next Steps

  • Continue working on the social extension, next week we want to have a functional prototype working.

Roadblocks

  • Could use a quick tutorial on template hooks. ie. How can my extension use a template hook in a Review Board template to add something to the page?

Dave

Status

  • Almost done better moved/renamed files project

Next Steps

  • Finish project, get more info about binary files project

Roadblocks

  • Strange Linux problems

Jim

Status

  • still working on diff rendering with descriptions and tests.
  • also investigating the bug that I discovered (comment gets out of sync between interdiff and summary)

Next Steps

  • continue what I am doing

Roadblocks

  • none for now

Steven

Status

  • Put basic web hooks extension up for review
  • Prototyped webhooks and reviewbot
  • Working on improving webhooks and getting reviewbot ready for review

Next Steps

  • Add support for repo cloning to SCM tools
  • Continue with reviewbot

Roadblocks

  • Would like some feedback on webhooks before I continue with current path

Wei Lu

Status

  • found and solved the problems in the code reused from r/2615
  • made further changes according to the comments in the thread

Next Steps

  • test it properly and debug, then post a review request to replace r/2615

Roadblocks

  • not sure how to test it on the local dev server properly

Wilson

Status

  • Made progress on project, near completion
  • More reading

Next Steps

  • Finish up and submit for review
  • Read more

Roadblocks

  • None

Yazan

Status

  • Working on required fields, almost done
  • Did some work on moving/enhancing summary table

Next Steps

  • Submit work for review

Roadblocks

  • None

preview-email/html/

Categories: Status Reports

Status Reports – February 4, 2012

February 4, 2012 Leave a comment

Anthony

Status
  • Fleshed out basic options for the other hooks (ReviewRequestActionHook,
    DiffViewerActionHook, working on ReviewRequestDropdownActionHook)
Next steps
  • I’m going to look into picking up another project
Roadblocks
  • Not sure if I should add this to the code review I already have out, so
    I’m waiting for some movement there

Steven

Status
  • Working on simple WebHooks extension
Next steps
  • ReviewBot
Roadblocks
  • None

Jim

Status
  • Trying to fix a bug that’s preventing me from checking-in previous
    project
  • Reading the diff utility code and see if I can re-use it and come up
    with a prototype for description change renderin
Next steps
  • Keep on what I am currently doing
Roadblocks
  • None

Yazan

Status
  • Working on moving summary table/adding buttons
  • Adding required fields to review request
Next steps
  • Submitting features for review
Roadblocks
  • None

Dave

Status
  • Better familiarized myself with code base of project scope (better moved
    files in diff viewer)
  • Planned and started coding project
Next steps
  • Continue working on project
Roadblocks
  • None

Curtis

Status
Next steps
  • Start the back end work required for the reviewboard-social extension, while the designs for the interfaces are reviewed and modified
Roadblocks
  • None

Wilson

Status
  • Sick, green phlegm everywhere
Next steps
  • Get better
Roadblocks
  • Nasal congestion

- Sorry I haven’t done much this week, not feeling well at all

Wei

Status
  • Made changes to r/2818 according to the comments and retested; got it pushed to master; closed r/2818
  • Starting project “Tracking User Changes” by reproducing the behavior on local test server
  • Downloaded and applied the starting code for project “Tracking User Changes” from previous review post r/2615
  • Researched the source code and diff for the project; learned django template syntax btw
Next steps
  • Started coding for the project
  • Hopefully posted a review request when early changes are done
Roadblocks
  • The project has been related to 8 files and 15 changes already; only intuitively understood the relationship between them so far; wish to get some extra help separately on their relationship
Categories: Status Reports
Follow

Get every new post delivered to your Inbox.