Status Report for October 25, 2014

Andrew Hong

What project are you working on?
Download Review Request File Attachments As ZIP, done? Tested and all, ready to go.
Starting to look at: Add support for matching commits to review requests to more commands.
And also looking at the users/groups autocomplete again.

What you accomplished this week?
Implemented suggestions and fixed minor bugs in zip feature, tested again and looks good to go now.
Started to look at other student projects that are still available. Revisiting aggressive autocomplete bug.

Links to anything you’ve done this week
https://reviews.reviewboard.org/r/6402/

What you plan to do next week?
Hopefully get to the bottom of the aggressive autocomplete bug. A couple of bug reports to base testing off of.
Also going to spend some time trying to add support for matching commits in rbtools.

What, if anything, is blocking you from making progress?
Nada!

Any other questions?
Nope.


Azad Salahli

What project are you working on?
‘rbt land’ for landing changes

What you accomplished this week?

  1. Figured out it is not possible (read: some non-trivial changes needed) to fix the API documentation where it says ‘Validate Diff List’. Apparently, anything that doesn’t have an id field in its URL should be declared as list.
  2. On my way to finish a prototype for ‘rbt land’ command. I’ll post a WIP until the meeting. Unfortunately it’s not ready yet.
    1. For now ‘rbt land’ takes a required request id and branch arguments, because all of the code for guessing request id is embedded inside ‘rbt post’ command and is not trivial to refactor.
    2. If the request doesn’t match the head of the specified branch, ‘rbt patch’ is used to apply the review request to that branch.
    3. The specified branch is pushed to remote repo as is (without being squashed).

Links to anything you’ve done this week
Posted https://reviewboardstudents.wordpress.com/2014/10/19/status-reports-for-october-18-2014/
Reviewed https://reviews.reviewboard.org/r/6393/
A [WIP] is on the way.

What you plan to do next week?
Working on my project. It’ll depend on the reviews to my [WIP] request.

What, if anything, is blocking you from making progress?
Nothing yet.

Any other questions?
None.


Barret Rennie

What project are you working on?
Currently I am working on the API caching layer for RBTools.

What you accomplished this week?
I’ve got an initial WIP draft done for the API caching layer. It currently doesn’t crash RBTools :D. It needs to undergo extensive testing and (probably) some refactoring to make it pretty. It is currently using SQLite as a database for the store. I’ve also added a review request for adding aliases to RBT.

I’ve also got a review request up for adding alias support to RBTools. It could use a review 🙂

Lastly I’ve cleaned up the review request for diff comment expansion and added a lot of testing info.

Links to anything you’ve done this week

Review Requests
https://reviews.reviewboard.org/r/6380/ (Diff fragment expansion)
https://reviews.reviewboard.org/r/6491/ (RBTools alias support)
https://reviews.reviewboard.org/r/6504/ (API Caching)

Hackpad for RBT Caching
https://reviewboard.hackpad.com/Barrets-Devlog-0FyARzq47OT#:h=Add-caching-to-RBTools-for-fas

Flowchart for (how I think) caching works
https://www.dropbox.com/s/s46rdynb33nbc9k/flowchart%20for%20caching.png?dl=0

What you plan to do next week?
Unfortunately, I won’t be able to accomplish much next week. I have two midterms to study for and I am moving next Saturday.

What, if anything, is blocking you from making progress?
Nothing, unless I misunderstand how HTTP caching is supposed to work. That stuff is complicated.

Any other questions?
Nope.


David Kus

What project are you working on?
Drag ‘n Drop inline images into the markdown editor

What you accomplished this week?

  1. Finished testing the backend
  2. Added unit tests to the new WepAPI Resource that I added
  3. Removed WIP tag from the review request for the backend work

Links to anything you’ve done this week.
https://reviews.reviewboard.org/r/6454/ (now ready for reviews) Bear with me as it’s a pretty big one.

What do you plan to do next week?
Get back into the frontend work (I had originally started there but decided that it would be best to have a working backend first). Need to see exactly what I have left to do there, but at least:

  1. Wire up the front-end to the WebAPI
  2. Look into adding a way to add review request file attachments while a markdown editor is open

What, if anything, is blocking you from making progress?
I am having some trouble setting up a branch for the frontend based on the backend. When I create a new branch based on my ‘backend’ branch, and then post a review request for this new branch, it contains all the backend changes as well. Is this expected? And is there some other way that I could create my new branch based off my ‘backend’ branch without this happening?

Any other questions?
Review bot doesn’t like my style, but I can’t seem to find another way to write my code. PyCharm’s pep8 tool isn’t marking this as an error, and I’ve seen the same style used in other test_* files. Any suggestions? (see https://reviews.reviewboard.org/r/6454/#comment17647)


Justin Malliet

What project are you working on?
I’m still working on sandboxing extensions.

What you accomplished this week?
I’m submitted my patch for sandboxing AuthBackend subclasses, and only just realized I’ll have to do more testing on it because it seems to be causing other tests to fail. I’ve got a [WIP] for djblets Column subclasses, which is mostly done except for 3 functions I still have to finish writing tests for. I tested this patch for the same issue making other tests fail and it does not seem to suffer the same problem. I’ve been doing some more research too, but i have not had a lot of time to update my Hacjkpad, so I’ll be doing most of that tomorrow.

Links to anything you’ve done this week
https://reviewboard.hackpad.com/Justy777s-Development-Log-PZgaCHPaopO
https://reviews.reviewboard.org/r/6399/
https://reviews.reviewboard.org/r/6502/

What you plan to do next week?
This coming week I plan to keep going in my progress and hopefully get as much of my current project done as possible.

What, if anything, is blocking you from making progress?
Nothing is blocking me for now.

Any other questions?
I have a few small questions, I’ll just ask during the meeting.


Mark Loyzer

What project are you working on?
Adding an extension to Review Board that will allow people to export a review request as PDF and XML.

What you accomplished this week?
I wasn’t able to do as much as I had hoped because I ran into some problems with finishing the implementation of the header and having 4 midterms to study for. But I did look into the Frame object in ReportLab (can view my conclusion on hackpad). Did fix the implementation of headers on every page to display Page X of Y. I also have started looking into and implementing how to input bullet points with indentations in the document.

Links to anything you’ve done this week
https://reviews.reviewboard.org/r/6413/ (I haven’t uploaded my changes yet because I want to do some work tomorrow on it and will upload them after that.)
Hackpad

What you plan to do next week?
Tidy up Header (still need to add the review request id to it).
Finish the Details section (should be quick once I get bullets working).
Add Issue Summary section.
Add File Summary section.

What, if anything, is blocking you from making progress?
Nothing

Any other questions?
None.


Ryan Done

What project are you working on?
File Attachment Diff Slider
Alphabetic Paginator

What you accomplished this week?
Alphabetic Paginator is finished and submitted
I spent some time last weekend addressing the visual design and making CSS tweaks with Mike’s help.

I started looking into writing a Spam Filter extension, but after talking with Christian for a while we decided that it would be too tricky to do without some big changes to reviewboard. So I started on adding a diff slider for file attachments (making use of the FileAttachementHistory class).

I should have a WIP review request up before I go to sleep tonight, but I didn’t want to hold up the status report collection for that.

Links to anything you’ve done this week.
https://reviewboard.hackpad.com/rdone-development-log-LK0GSpfQxxN
https://reviewboardstudents.wordpress.com/2014/10/19/meeting-minutes-oct-19-2014/
https://reviews.reviewboard.org/r/6369/ (submitted)
https://reviews.reviewboard.org/r/6368/ (submitted)

What do you plan to do next week?
Dig into the File Attachment diff slider. I don’t foresee it taking that much more time.

What, if anything, is blocking you from making progress?
All good for now.

Any other questions?
Nope.


Nicole Xin

What project are you working on?
Same as last week, general comments

What you accomplished this week?

  • I’ve written 106 tests for general comments web api, spent a lot of time debugging, from failing all to passing all. That’s huge!
  • Good news is I’m done on api and move on to front end.
  • Small modifications to backend model: add a get_absolute_url() function
  • Front end model built, templates todo

Links to anything you’ve done this week

What you plan to do next week?

  • brennie just point out a typo in my model, got to fix it
  • Finish templates for front end
  • Write tests for front end and do the testing
  • Split all my modifications on file attachment comment to a new review request

What, if anything, is blocking you from making progress?
I have trouble rebasing api on my comment_model(backend) branch. Brennie suggest manually merge, it seems that I even made it worse. Now I got a normal backend branch, and I “normal”(fixed to pass all tests) api branch. But if I try to rebase api on model, there’s tons of conflicts almost every commit applies. Hopefully I can solve this tomorrow.

Any other questions?
Nope.

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