Category Archives: Status Reports

[Open Academy] Status Reports: May 2, 2015

Jessica Qian

  • What project are you working on?
    • Trivial publishes
  • What you accomplished this week
    • Fixed bug with trivial being True.
  • What you plan to do next week
    • Work on final report requirement for class.
  • What, if anything, is blocking you from making progress?
    • None

  • Any other questions
    • None

Teresa Fan

  • What project are you working on?
    • Better Linking in the UI
  • What you accomplished this week
    • Changed the link icon to be in a different place (now inline with the comment text, and right-aligned)
  • What you plan to do next week
    • Adding link icons for replies too?
  • What, if anything, is blocking you from making progress?
    • Nothing. Reviews are always welcome
  • Any other questions
    • Nope

Viet-Tran Nguyen

  • What project are you working on?
    • Pre-post hook and uncommitted flag
  • What you accomplished this week
    • Address some left over comments for the above issues
  • What you plan to do next week
    • Work on final report requirement for class.
  • What, if anything, is blocking you from making progress?
    • None
  • Any other questions
    • None

[Open Academy] Status Reports: May 10, 2015

Teresa Fan

  • What project are you working on?
    • Better Linking in the UI
  • What you accomplished this week
    • Nothing really, see below
  • What you plan to do next week
    • My semester is over, but I’d like to finish this project (even if it’s not “graded”), so whatever is needed.

  • What, if anything, is blocking you from making progress?
    • I had finals all this week (classes actually ended last week) so didn’t get much chance to work on this. I’ll have a lot more time now to do whatever is needed to finish it though! I don’t mind if it’s not evaluated.

  • Any other questions
    • What next?

[Open Academy] Status reports – April 11th, 2015


Tran Nguyen

1.What project are you working on?
Extension framework testing

2. What you accomplished this week.
Discuss with Christian what needs to be done, start looking at extension
manager, install rbtmod extension and trace point

3. Links to anything you’ve done week:
Nothing new except the two pending CR
https://reviews.reviewboard.org/r/6879/
https://reviews.reviewboard.org/r/7066/

4. What you plan to do next week.
Gather the remaining knowledge to start implementing extension testing
framework

5. What, if anything, is blocking you from making progress?
Nothing yet, I will ask on Slack.

6. Any other questions.
Nope.



Teresa Fan

1.What project are you working on?

Interactive Issue Summaries

2.What you accomplished this week.

Finally fixed my bug so that now issue statuses update in both places correctly regardless of where you update them.

Wrote tests for this.

3.Links to anything you’ve done week

https://reviews.reviewboard.org/r/6997/

https://reviewboard.hackpad.com/Teresas-Notes-gI6ljBKu7ra

4.What you plan to do next week.

Next steps originally included adding a link with the number of replies that brought you to the comment, but maybe that’s not necessary anymore since we have the ‘See comment’ link. I also thought about making the cell expand on hover instead of click.

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

If anyone could review my code that’d be great!

6.Any other questions.

Asking in Slack about git problems. See #4 for questions about what would be best to do next.



Rohan Meringenti

1.What project are you working on?

Revokable Ship it Labels

2.What you accomplished this week.

Finallly fixed my bug with the event listeners. You can now see the changes immediately on the page. Furthermore, the ship it counter has also been updated for the review request.

3.Links to anything you’ve done week

https://reviews.reviewboard.org/r/7170/

4.What you plan to do next week.

Finish style, add tests. and pick a new task

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

If someone could look at my WIP that would be great. All the big changes are already there, I just have to add style and tests now.

6.Any other questions.

Nope



Chen Feng

1.What project are you working on?

Create a new plugin.

2.What you accomplished this week.

reset my environment, and fixed a bug.

3.Links to anything you’ve done week

https://reviews.reviewboard.org/r/7191/

https://reviewboard.hackpad.com/Fengs-Note-lHTmrx16YQd

4.What you plan to do next week.

Download Review Bot. Write my project.

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

NO

6.Any other questions.

No~ if have, I’ll ask it in slack.



Sihang Chen

  1. What project are you working on?

Add unreviewed section to dashboard.

  1. What you accomplished this week.

Accomplish the function to show the unreviewed review requests, and the count number of outgoing unreviewed review requests, but still not many tests.

  1. Links to anything you’ve done week

hackpad: https://reviewboard.hackpad.com/Chen-Sihangs-Notes-1t6Ki8onOzy

reviewboard_reviewrequests: https://reviews.reviewboard.org/r/7193/

  1. What you plan to do next week.

Continue working on the incoming count number.

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

School work of my own.

  1. Any other questions.

Not yet, but please review my review requests at the link above, thank you so much ^_^.



Chen Mingyi

  1. What project are you working on?

Add infinite scrolling to the diff viewer

Read-only site mode

  1. What you accomplished this week.

Fix issues and start a new project.

  1. Links to anything you’ve done week

https://reviews.reviewboard.org/r/7047/

https://reviewboard.hackpad.com/Mingyis-Notes-gvT8udhK8GY

  1. What you plan to do next week.

To stop some actions.

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

How many functions do review board have, and how many should be stopped.

  1. Any other questions.

The previous project is still waiting for review!



Chenxi Ni

1) What project are you working on?
OAuth2 provider support

2) What you accomplished this week.

looking into how OAuth2 works and follow the tutorial and try
3) Links to anything you’ve done this week (hackpads, review requests,
blog posts, etc)

My RR for my first project:  https://reviews.reviewboard.org/r/6990/

My hackpad:https://reviewboard.hackpad.com/Chenxis-Notes-hRtQP9sWSrP
4) What you plan to do next week.

according to the tutorial to list what should be done and write more basic code.
5) What, if anything, is blocking you from making progress?

No.
6) Any other questions

Not yet.



Sun Wang Jun

  1. What project are you working on?

Checklist extension (templates)

  1. What you accomplished this week.

Refactored the checklist code base and updated the review request

  1. Links to anything you’ve done week

https://reviews.reviewboard.org/r/7176/

  1. What you plan to do next week.

Code the ui for templates

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

School projects

  1. Any other questions.

No



Stanley Yeo

  1. What project are you working on?

Improve Review Bot’s admin interface

Auto-add users to groups

  1. What you accomplished this week.

Wrote some documentation.

Test functionalities.

Update RR.

  1. Links to anything you’ve done week

https://reviews.reviewboard.org/r/7176/

  1. What you plan to do next week.

https://reviews.reviewboard.org/r/7165/

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

None

  1. Any other questions.

Please review 6998, 6999, 7165



Xuanyi

  1. What project are you working on?

Framework and UI for integrating with third-party services

  1. What you accomplished this week.

Update review request

Implement basic RBslack with integration.

  1. Links to anything you’ve done week

https://reviews.reviewboard.org/r/7105/

https://reviews.reviewboard.org/r/7136/

  1. What you plan to do next week.

Continue to work on integration form and rbslack with integration

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

Some questions on one of my review: https://reviews.reviewboard.org/r/6918/

School projects.

  1. Any other questions.

The rbslack integration could potentially be use to target specific group/user. However, will it be possible for subclass of SiteSettingForm to support autocomplete?



Jifei Wen

1.   What project are you working on?
reviewbot plugin PEP257 and shellcheck

2.   What you accomplished this week.
PEP257: fix bug and modify docstring shellcheck: write it

3. Links to anything you’ve done week:

https://reviews.reviewboard.org/r/7101/

https://reviews.reviewboard.org/r/7197/

4. What you plan to do next week.

modify both plugins

5. What, if anything, is blocking you from making progress?
no, i will ask at slack or reviews

6. Any other questions.
No.



Wu Di

1. What project are you working on? 
Allow replying to reviews from the diffviewer

2. What you accomplished this week. 
Got the updating of draft review replies and making new review replies to work properly.

Added in styling for the review reply editor.

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

4. What you plan to do next week. 
Continue working on getting the comment blocks to update its serialized comments upon adding or updating replies.

5. What, if anything, is blocking you from making progress? 
Would appreciate feedback on the current UI. (https://reviews.reviewboard.org/r/7172/file/891/)

Also, some reviews for my other two review request would be nice, https://reviews.reviewboard.org/r/6970 and https://reviews.reviewboard.org/r/7032.

I will be busy with my other school projects this week.

  1. Any other questions. 

Is there a good way to publish the review reply from the diffviewer? Christian suggested a while back to have a banner at the top of the page, similar to the draft review banner, but that would be confusing.

Or is it even a good idea to allow publishing/discarding of review replies in the diffviewer when the other replies of the review reply cannot be seen?

For example, you have an incomplete body top reply along with a diff comment reply, publishing the review reply will make both replies public, which is bad.

I think it isn’t very obvious that review replies are basically reviews where you make a bunch of replies to the body top/diff/file attachment comments of another review, this might cause users to publish a review reply thinking that they are only publishing one diff comment reply instead of all the other replies in the review reply.



Jessica Qian

1.What project are you working on?

“Trivial” reviews
2. What you accomplished this week.
I was working a poster presentation for this class.
3. Links to anything you’ve done week:
https://reviewboard.hackpad.com/Jessicas-Hackpad-LOYsF3tQwem
https://reviews.reviewboard.org/r/7022/
https://reviews.reviewboard.org/r/7022/

4. What you plan to do next week.
I need to clarify the UI details for trivial reviews.

Respond to RR for trivial review requests.

5. What, if anything, is blocking you from making progress?
None.

[Open Academy] Status reports – April 4th, 2015


Mingyi Chen

1. What project are you working on?
Add infinite scrolling to the diff viewer

2. What you accomplished this week.
Fix issues and push release on the review board.

3. Links to anything you’ve done week
https://reviews.reviewboard.org/r/7047/
https://reviewboard.hackpad.com/Mingyis-Notes-gvT8udhK8GY

4. What you plan to do next week.
Waiting for evaluation. And try to make it better.
Finish the project, select another:)

5. What, if anything, is blocking you from making progress?
No.

6. Any other questions.
Waiting now ~:)
I have dropped some issues, and post it again.Wait for it~



Rohan Meringenti

1. What project are you working on?
Removable Ship It Links

2. What you accomplished this week.
Got event listeners to listen to events and facilitate loading of page in api resource

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

4. What you plan to do next week.
Add styling for new ship it label
write test cases for several situations in which deleting ship it labels

5. What, if anything, is blocking you from making progress?
None

6. Any other questions.
None



Wu Di

1. What project are you working on?
Allow replying to reviews from the diffviewer

2. What you accomplished this week.
Got one part working, replies can be made to another reply in the comment editor

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

4. What you plan to do next week.
Add styling for the inline editor in the comment editor
Continue working on getting the inline editor to work properly

5. What, if anything, is blocking you from making progress?
None

6. Any other questions.
None



Sun Wang Jun

1. What project are you working on?
Checklist extension (integrating templates)

2. What you accomplished this week:
Began reworking of existing code to integrate templates

3. Links to anything you’ve done this week:
https://reviewboard.hackpad.com/2015-Checklist-Extension-oLIpxP1SzyX

4. What you plan to do next week:
Finish up reworking of api to allow nested items
Start changing UI to include templates

5. What, if anything, is blocking you from making progress?
Projects for other modules, but should not affect much
Reviews for my requests will be nice: https://reviewboard.hackpad.com/Suns-Notes-CSvDSCiw0Lo#:h=My-Review-Requests-:heart:

6. Any other questions?
Nope



Jessica Qian

1. What project are you working on?
‘Trivial’ publishes

2. What you accomplished this week:
Replied to comments on RR

3. Links to anything you’ve done this week:
https://reviewboard.hackpad.com/Jessicas-Hackpad-LOYsF3tQwem
https://reviews.reviewboard.org/r/7022/
https://reviews.reviewboard.org/r/7125/

4. What you plan to do next week:
We have poster presentations for this class, so I will mainly be working on that.

5. What, if anything, is blocking you from making progress?
The UI details for trivial reviews are unclear.

6. Any other questions?
None



Stanley Yeo

1. What project are you working on?
Auto-add users to groups

2. What you accomplished this week.
Demo video
DB migration to add “is_default” field to group
Modify admin setting to show the “is_default” field on admin site
Write signal handler to handle a new user of a local site
Test
Post a Review Request.

3. Links to anything you’ve done week
https://reviews.reviewboard.org/r/7165/
https://reviewboard.hackpad.com/Stanleys-Note-y91i8NygfSk

4. What you plan to do next week.
Update RR

5. What, if anything, is blocking you from making progress?
None

6. Any other questions.
None



Teresa Fan

1. What project are you working on?
Interactive Issue Summaries

2. What you accomplished this week.
Investigated why buttons are disabled when an issue is reopened (see #4).

3. Links to anything you’ve done week
https://reviews.reviewboard.org/r/6997/
https://reviewboard.hackpad.com/Teresas-Notes-gI6ljBKu7ra

4. What you plan to do next week.
Still working on the bug where when an issue is reopened after it was dropped/fixed in-table, the button bar doesn’t reappear correctly in the table (they appear disabled). I have to do my final presentation for the course I’m taking for this at school in 2 weeks, so I’m going to try my best to finish up this project.

5. What, if anything, is blocking you from making progress?
The bug mentioned above.

6. Any other questions.
Asking in Slack.



Tran Nguyen

1. What project are you working on?
Looking for new project

2. What you accomplished this week.
Address comments from old PR.

3. Links to anything you’ve done week
https://reviews.reviewboard.org/r/6879/
https://reviews.reviewboard.org/r/7066/

4. What you plan to do next week.
Find a new project

5. What, if anything, is blocking you from making progress?
No comments on my PR.

6. Any other questions.
No.



Xuanyi Lin

1. What project are you working on?
Framework and UI for third-party services

2. What you accomplished this week:
Updating the reviews according to the feedback
Integration forms and config template

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

4. What you plan to do next week:
Continue to work on integration forms and config template
Start to implement a functional integration.

5. What, if anything, is blocking you from making progress?
Removing load method from integration is causing some initialisation issue currently.

6. Any other questions?
Will be asking in the reviews and Slack.



Chenxi Ni

1) What project are you working on?
Changing Review Request Ownership

2) What you accomplished this week.
Finish the project and find Add OAuth2 provider support as next.

3) Links to anything you’ve done this week (hackpads, review requests,
blog posts, etc)
My WIP RR for my feature: https://reviews.reviewboard.org/r/6990/
My hackpad:https://reviewboard.hackpad.com/Chenxis-Notes-hRtQP9sWSrP

4) What you plan to do next week.
look into how OAuth2 works and try to list what should do

5) What, if anything, is blocking you from making progress?
No.

6) Any other questions
Not yet.

[UCOSP] Status Report Mar 28

Vincent Le

1) What project are you working on?

    – FileProvider framework

2) What you accomplished this week.

    – Made changes as suggested by reviews

    – Added a `api/file-providers/` end-point to make API calls to know what file providers are supported

    – Added backbone.js model for file-providers from api calls to `api/file-providers/`. Integrated with the upload modal to use information needed for progression

    – Re-did the design for display of metadata, moving logic to files where needed and display path only for search results.

    – Added search logic to show search results on the options for selection. Search result is displayed after 500ms after the last key up in the search field, or display the current path when search field is white space

    – more variable refactoring as names were missed

    – made some reviews

3) Links to anything you’ve done this week:

    – FileProvider WebApi interaction with file provider account: https://reviews.reviewboard.org/r/6948/

    – [WIP] FileProvider Modal UI for uploads: https://reviews.reviewboard.org/r/7009/

    – FileProvider resource web API for providers: https://reviews.reviewboard.org/r/7124/

    – Reviews: https://reviews.reviewboard.org/users/VTL-Developer/reviews/

4) What you plan to do next week.

    – Clean up and finish r/7009 (test cases, making changes as response to reviews, etc.)

    – Demo Video

5) What, if anything, is blocking you from making progress?

    – N/A

6) Any other questions

    – N/A

Tien Vu

1. What project are you working on?

Diff Doctor

2. What you accomplished this week.

– Not sure what else to do for my 2 projects, so I went around for code reviews and bug fixing instead

3. Links to anything you’ve done:

https://reviews.reviewboard.org/r/6833/

https://reviews.reviewboard.org/r/6993/

Code reviews:

https://reviews.reviewboard.org/r/6879/

https://reviews.reviewboard.org/r/7066/

Learned about git hooks: http://git-scm.com/book/en/v2/Customizing-Git-Git-Hooks

and fixed the bug at https://code.google.com/p/reviewboard/issues/detail?id=3824. Review request link:

https://reviews.reviewboard.org/r/7129/

4. What you plan to do next week.

– I need some direction here. What should I do?

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

– What should I do if patching fails to something outside the patch() function (that throws a PatchError)? Like an SCMError.

– What else do I have to do to land my projects?

No luck with reproducing these bugs:

https://code.google.com/p/reviewboard/issues/detail?id=3108

https://code.google.com/p/reviewboard/issues/detail?id=3421

https://code.google.com/p/reviewboard/issues/detail?id=3687

6. Any other questions.

– No

Jessica Yuen

1. What project are you working on?

  • rb-gateway

2. What you accomplished this week.

  • Unit testing for all rb-gateway API’s, testing:
    • /repos/{repo}/file{id}, GET, HEAD
    • /repos/{repo}/commits/{id}/path/{path}, GET, HEAD
    • /repos/{repo}/path, GET
    • /repos/{repo}/branches, GET
    • /repos/{repo}/branches/{name}/commits, GET
    • /repos/{repo}/commits/{id}, GET
    • /session, GET
  • Minor fix to return 30 commits vs 20, to align with GitHub API, landed.
  • Addressed issues with 7053, landed

3. Links to anything you’ve done week:

4. What you plan to do next week.

  • Demo video
  • Reviews
  • Address any reviews on outstanding review request, 7134, get it landed.

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

  • N/A

6. Any other questions.

  • Nope


Chester Li

1) What project are you working on?

  • Bug fixing and code reviewing

2) What you accomplished this week.

  • Landed the main project.
  • Reviewed code changes.
  • [WIP] bug fixing.

3) Links to anything you’ve done this week:

4) What you plan to do next week.

  • Recorded the demo and hopefully fixed issue 3811

5) What, if anything, is blocking you from making progress?

  • Was overwhelmed by other projects and homework

6) Any other questions

  • N / A


Christopher Arnold

1) What project are you working on?

  • Mute and Archive Review Requests

2) What you accomplished this week.

  • split up project into smaller review requests
  • finished automatic unarchiving
  • fixed URL query
  • finished ReviewRequestVisitTests in accounts.models
  • finished test_show_archived in datagrids.grids
  • wrote most of test_archived_review_request for the webapi
  • wrote most of reviewRequestEditorViewTests in static.rb.js.views

3) Links to anything you’ve done this week:

4) What you plan to do next week.

  • fix the last two tests mentioned above
  • video demo
  • polish documentation
  • do some reviews

5) What, if anything, is blocking you from making progress?

  • test_archived_review_request for the webapi: https://reviews.reviewboard.org/r/7087/diff/7/#6
    • I’m getting the same error for 3 of the DELETE tests: AssertionError: u’A’ != u’V’
    • When I print the visibility in the actual delete function, it is as expected (‘V’), but for some reason doesn’t show up like that in the test
  • I actually break 4 of the original tests (all similar) with my patch, can’t figure out how to fix my code
    • error is with setStarred: Expected spy success to have been called
      • on lines 46 and 65 of reviewGroupModelTests
      • and on lines 167 and 187 of reviewRequestModelTests
    • I think it has to do with how I implement addImmediately and removeImmediately when adding the trigger.

6) Any other questions

  • How complete is my project looking? Any more tests I should do?
  • Super stuck for a while now on the things I mentioned above, so I would appreciate any help 🙂 Thank you!


Jason Tseng

1. What project are you working on?

Responsive UI

2. What you accomplished this week.

– Updates on part 1 and rebased through the rest

– secondary menu button design

3. Links to anything you’ve done week:

– https://reviews.reviewboard.org/r/6796

– https://reviews.reviewboard.org/r/6923/

4. What you plan to do next week.

– Demo

– Minor bug fixes

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

– n/a

6. Any other questions.

– Nope.

[UCOSP] Status Reports: March 15th to 21st


Vincent Le

1) What project are you working on?
– FileProvider framework

2) What you accomplished this week.
– Made changes as suggested by Christian and Barret in my reviews
– Added search logic to show search results on the options for selection. Search result is displayed after 500ms after the last key up in the search field, or display the current path when search field is empty.
– Started implementing display metadata for selected files
– Did some small refactoring for my JS file
– Fixed a bug regarding metadata in my test cases and testing class for FileProviders

3) Links to anything you’ve done this week:
– FileProvider module implementation with Django : https://reviews.reviewboard.org/r/6806/
– FileProvider WebApi integraton : https://reviews.reviewboard.org/r/6948/
– [WIP] FileProvider Framework UI : https://reviews.reviewboard.org/r/7009/

4) What you plan to do next week.
– Finish display metadata
– Make my file navigation for FileProvider modal compatibile with any provider
– Get to create accounts

5) What, if anything, is blocking you from making progress?
– Nothing I can think of yet

6) Any other questions
– Nothing I can think of yet


Chester Li

1) What project are you working on?
  • Post-commit review request support for GitLab


2) What you accomplished this week.
  • Refactored code for 2 review requests according to reviews.
  • One bug fix has been submitted
3) Links to anything you’ve done this week:
4) What you plan to do next week.
  • Start a new project, do as much as I can
5) What, if anything, is blocking you from making progress?
  • I was basically waiting for the review and did some changes.
6) Any other questions
  • Not yet


Tien Vu

1) What project are you working on?
    – Diff Doctor
2) What you accomplished this week.
    – In summary, replaced failed patch traceback with inline Pygments, as collapsible links. I tried to figure out how to make it like the plus button for a review request change, but I lack the frontend knowledge, so I’m sticking with <a href>.
3) Links to anything you’ve done this week:
    – Learned how to use Pygments: http://pygments.org/docs/
4) What you plan to do next week.
    – Decide what to do when patch fails because of a non PatchError (like SCMError). Right now it just links to a template that displays the error like {{ error.msg }}.
    – After this, is there anything major left to do for my 2 projects? For Diff Doctor, Christian suggested me to skip “step 2” of analyzing the source problem of failing the patch, but we agreed to skip over this because it’s too complicated for now. The other 2 parts of showing the .zip and showing the reject files instead of the traceback works. Should I pick a smaller project to work on?
5) What, if anything, is blocking you from making progress?
    – How can I land my changes to master? Should I at least push my branch(es) up?
6) Any other questions
    – No


Jessica Yuen

1. What project are you working on?
rb-gateway
2. What you accomplished this week.
Sorry I’ve been a super occupied with other work this week…
– Fixed outstanding issues on existing review requests (7053 & 7054)
– Added unit testing for configuration loading

3. Links to anything you’ve done week:
4. What you plan to do next week.
– Since Pencils Down is a week and half from now, I think I’ll just concentrate on adding any missing test cases, getting rb-gateway to a polished state. I think it’s already pretty complete for the features that have been requested so far, but let me know if I’m wrong.

5. What, if anything, is blocking you from making progress?
– N/A

6. Any other questions.
– Nope.


Chris Arnold

1) What project are you working on?
  • Mute and Archive Review Requests
2) What you accomplished this week.
  • handled automatic unarchiving
  • fixed URL query
  • split up project into smaller review requests
3) Links to anything you’ve done this week:
4) What you plan to do next week
  • attend to several reviews by David and Barret
  • getting used to the workflow of using several review requests that depend on each other
  • tests
5) What, if anything, is blocking you from making progress?
  • the splitting up of my project
6) Any other questions
  • I’ve split up my project, but it’s been trickier than I thought, especially when trying to decide which line of code belongs to which review request… Working on my project seems a lot more fragile now, and my pace has definitely slowed down. It seems like I need to rebase AND rbt post up to 3 branches whenever I want to make a change, since all my review requests rely on each other.
    • Any tips out there that I could use to make this process easier?
    • Is there a slick way to transfer changes from one review request to another?


Jason Tseng

1. What project are you working on?

Responsive UI

2. What you accomplished this week.

– Additional aesthetic update on part 1

– Changes applied to the rest

– half way on secondary menu button design

3. Links to anything you’ve done week:

– https://reviews.reviewboard.org/r/6796

4. What you plan to do next week.

– Finish secondary menu button

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

– passing review on part 1

6. Any other questions.

– Nope.

[Open Academy] Status reports – March 14, 2015

Chenxi Ni

1) What project are you working on?

Changing Review Request Ownership

2) What you accomplished this week.

Finish adding some notifications to users before they publish with different submitter.

3) Links to anything you’ve done this week (hackpads, review requests,

blog posts, etc)

My WIP RR for my feature:  https://reviews.reviewboard.org/r/6990/

My hackpad:https://reviewboard.hackpad.com/Chenxis-Notes-hRtQP9sWSrP

4) What you plan to do next week.

Start with adding tests.

5) What, if anything, is blocking you from making progress?

Not yet.

6) Any other questions

Not yet.

Teresa Fan

1. What project are you working on?

Interactive Issue Summaries

2. What you accomplished this week.

Addressed comments on my review request.

3. Links to anything you’ve done week

https://reviews.reviewboard.org/r/6997/

4. What you plan to do next week.

Get the buttons to actually work, I haven’t properly hooked them up with the commentManagerModel.

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

Nothing for now, spring break is over so I’ll be back to making real progress.

6. Any other questions.

How do I remove a diff file from a review request? My review request mistakenly includes the .DS_Store file.

Sun Wang Jun

1. What project are you working on?

Checklist extension (user templates)

2. What you accomplished this week:

Updated checklist extension UI review request

Understood WebAPIResource, Backbone.js

Started work on templates

3. Links to anything you’ve done this week:

https://reviews.reviewboard.org/r/6969/ // updated review request

4. What you plan to do next week:

Finish up user templates

Start thinking about integration with the current ui

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

Extension keeps shutting down after a while, I know when it happens but I don’t know why it happens:

a) The logger restarts

b) Then the extension shuts down, and restarts

c) But the AccountPagesHook no longer hooks into account pages

which means I have to manually restart the entire server to get the hook working again

6. Any other questions?

I think I asked in Slack at inappropriate timings… so:

a) I think it will be less confusing if checklist templates were to be called checklist groups, or something else. Any opinions?

b) I can live with the extension shutting down problem for now, just wastes a few seconds everytime, or should I focus on fixing that first?

c) In the checklist templates/groups, is there a need for the checklist items to be checked by default? I can think of a possible use case as checking all items by default, so that the user only unchecks.

And a misc question: Does WebAPI not allow or deal with XML request payloads in incoming HTTP requests? I wanted to use Backbone.Collection.create but that sends POST request data as Request Payload instead of Form Data, as a result the @webapi_request_fields was not able to extract the fields.

  

Xuanyi Lin

1. What project are you working on?

Framework and UI for integrating with third-party services

2. What you accomplished this week:

Implemented webapi for Integration and ConfguredIntegration model

Implemented the backbone model and template

Added resource path for icon from integration

Partially done admin page for integrations

Fix issues for my previous RR

3. Links to anything you’ve done this week:

RR for part 1 of my project: https://reviews.reviewboard.org/r/6918/

RR for part 2 of my project: https://reviews.reviewboard.org/r/7034/

Code review: https://reviews.reviewboard.org/r/6999/

4. What you plan to do next week:

Finish up on the backbone and template for integrations

Split the current RR for part 2 into two parts

Work on test for part 2 if possible

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

Nothing much for now.

6. Any other questions?

Nope.

Wu Di

1. What project are you working on?

Better Diff Comment Bubbles

2. What you accomplished this week:

Implemented all the functionalities of the third part of the project to make the draft comments movable and resizable.

Updated the first project based on review comments.

3. Links to anything you’ve done this week:

Hackpad: https://reviewboard.hackpad.com/Wu-Dis-Notes-sULvWjtXN56

Part 1: https://reviews.reviewboard.org/r/6932/

Part 3: https://reviews.reviewboard.org/r/7032/

4. What you plan to do next week:

Review code.

Update my review requests when necessary.

Start looking for a new project.

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

Some code reviews for part 2 (https://reviews.reviewboard.org/r/6970/) would be nice.

6. Any other questions?

I’ve finished implementing the moving and resizing functionalities based on Christian’s comments, but I’m not sure whether it’s correctly done.

Part 2 of my project modifies the same file textBasedCommentBlockView.js as part 3 of the project, they are independent of each other but should I integrate them now?

Rohan Meringenti

1. What project are you working on?

Highlighting Active Issues

2. What you accomplished this week:

Made front end changes to site to reflect new issues

Started backend for grouping issues. Trying to come up with

some ideas for how to reflect the changes in the front end to be ui friendly.

3. Links to anything you’ve done this week:

https://reviews.reviewboard.org/r/7008

4. What you plan to do next week:

Start looking for a new project

Review code

Finish part 2 of the project

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

I’ll put up an update to front end when I get home today. I had a question on how less files were being generated. I updated one of the less

files to add a class tag with a style elements. However, when I add the tag in the html the changes aren’t there, do I have to compile the less files for the

new changes to show up? I’m not really familiar with how less works.

6. Any other questions?

If I wanted to post screenshots for how the page looks with the front end changes, how would I be able to do that in a review?

Stanley Yeo

1. What project are you working on?

Improve Review Bot’s admin interface

2. What you accomplished this week:

Updated RRs

3. Links to anything you’ve done this week:

https://reviews.reviewboard.org/r/6998/

https://reviews.reviewboard.org/r/6999/

4. What you plan to do next week:

Finish current project.

Pick another project to work on.

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

None.

6. Any other questions?

None.

Jessica Qian

1. What project are you working on?

“Trivial” publishes

2. What you accomplished this week:

Trivial publishes work completely for review requests.

3. Links to anything you’ve done this week:

https://reviews.reviewboard.org/r/7022

https://reviewboard.hackpad.com/Jessicas-Hackpad-LOYsF3tQwem

4. What you plan to do next week:

Not much, because it is midterm week again.

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

None.

6. Any other questions?

Nope.

Mingyi Chen

1. What project are you working on?

Add infinite scrolling to the diff viewer

2. What you accomplished this week:

Test for scroll bugs and fix.

3. Links to anything you’ve done this week:

https://reviews.reviewboard.org/r/7047

https://reviewboard.hackpad.com/Mingyis-Notes-gvT8udhK8GY

4. What you plan to do next week:

Finish the project

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

Scroll to the bottom and scroll up, it will still recognize it at the bottom.

6. Any other questions?

No. 🙂

Viet-Tran Nguyen

1. What project are you working on?
 – adding pre-post hook

2. What you accomplished this week:
 – prototype to run the command specified in .reviewboardrc

3. Links to anything you’ve done this week:
 – still in my local branch

4. What you plan to do next week:
 – Finish the project

5. What, if anything, is blocking you from making progress?
bottom.
– Midterm

6. Any other questions?
No.