Author Archives: seanchain1996

[Open Academy] Status reports – March 28th, 2015


Sihang Chen

1. What project are you working on?
Add the unreviewed section to the dashboard.
2. What you accomplished this week:
Actually, I’ve been busy preparing some documents for a competition, so I’ve got little time to do the reviewboard thing, and all I’ve done was to try to figure out how to automatically add the two additional columns in the database when building the project automatically.
3. Links to anything you’ve done this week:
No update to my hacked, the links to last week:
https://reviewboard.hackpad.com/Chen-Sihangs-Notes-1t6Ki8onOzy
4. What you plan to do next week:
Next week I’ll have more time and try to work out the database thing.
5. What, if anything, is blocking you from making progress?
I still have the school work to do and another project to work on, but I’ll accomplish what I’m supposed to finish for review board, that’s for sure.
6. Any other questions?
No, if any, I’ll ask on Slack.


Teresa Fan

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

2. What you accomplished this week.
Fixed a minor bug where the button divs where rendering in two places (although one was left empty).
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).

5. What, if anything, is blocking you from making progress?
The bug mentioned above, but I’ll put a little more time into it before I really say I’m stuck.

6. Any other questions.
I think the problem is that I have 2 separate commentIssueBars, one that I made for the table and the other that was existing for the place in the actual review. How do I get my new bar to “listen” to the other bar so that it properly updates the in-table status to “open” when it’s reopened from outside the table?


Jessica Qian

1. What project are you working on?
“Trivial” publishes

2. What you accomplished this week.
Recorded the demo.
Finished backend for trivial reviews.

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

4. What you plan to do next week.
I will work on frontend for trivial reviews.

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

6. Any other questions.
None.


Mingyi

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

2. What you accomplished this week.
Record the video
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.

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

6. Any other questions.
Waiting now ~:)


Chenxi

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

2) What you accomplished this week.
add all tests.
now correcting code by comments

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.
Finish the project and find another.

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 (integrating templates)

2. What you accomplished this week.
Video demo.
Fixed a bug and improved UI for stuff in current review requests.

3. Links to anything you’ve done week
https://reviewboard.hackpad.com/2015-Checklist-Extension-oLIpxP1SzyX#:h=Integration
https://drive.google.com/file/d/0ByHWCSTdXEMLMkFHUUlWbWxqeWs/view

4. What you plan to do next week.
Most probably re-working the checklist from ground up to support templates

5. What, if anything, is blocking you from making progress?
Contemplating whether to just modify the existing code or to re-design checklists

6. Any other questions.
What are the things we should look out for before pressing Ship It for others’ review requests, assuming we are allowed to do so?


Jifei

1. What project are you working on?

PEP257 ReviewBot tool
2. What you accomplished this week:

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

https://reviews.reviewboard.org/r/7101/
4. What you plan to do next week:

complete the docstring and perfect the PEP257
5. What, if anything, is blocking you from making progress?

None

6. Any other questions?

i can not find the entry point at `reviewbot.tool`


Rohan Meringenti

1. What project are you working on?
Revokable Ship its

2. What you accomplished this week.
Finished Backend of removable ship its
Working on unit tests

3. Links to anything you’ve done week
Nothing right now, Updating with demo video later today.

4. What you plan to do next week.
Update ship it counter and republish the page after the ship it model changes

5. What, if anything, is blocking you from making progress?
I posted this in #students last night, but I was confused as to what .publish() method does. I need the page to refresh
after the model is loaded. I was looking at how the ship it button worked here,(this calls the publish method), but im not sure how
to replicate the reload.

6. Any other questions.
nope.


xuanyi

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

2. What you accomplished this week.
Added Webapi Resource test
Implemented integration form
Updated Integration class

3. Links to anything you’ve done week
https://reviews.reviewboard.org/r/6918/
https://reviews.reviewboard.org/r/7105/
https://reviews.reviewboard.org/r/7106/
https://reviews.reviewboard.org/r/7136/ [WIP]

4. What you plan to do next week.
Continue to work on integration form and config template

5. What, if anything, is blocking you from making progress?
Will be great to have some feedback for my previous RR so that I could improve on it.

6. Any other questions.
Currently, I am trying to make use of the IntegrationForm(a subclass of SiteSettingsForm) to provide the form field for changing the description of an ConfiguredIntegration in the configure page. However, the description field exist as a model field rather than within the json field of the model, so I will need to update them differently. Is there any better way of doing this?


wudi

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

2. What you accomplished this week:
Haven’t made much progress this week, I was reading up the codebase on how the components talk together to create a review reply.
Got the reviews panel in the comment editor dialog to display the rendered comment and reply hierarchy.
Fixed a bug in my review request after updating with the changes in master.

3. Links to anything you’ve done this week:
https://reviews.reviewboard.org/r/7032/
https://reviewboard.hackpad.com/Wu-Dis-Notes-sULvWjtXN56

4. What you plan to do next week:
I have a rough idea of how to simplify the current UI design based on Christian’s reply, will proceed with it for now and see how it turns out. I’ll try to get a working prototype of the review reply editor by end of the week.

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

6. Any other questions?
No.


Viet-Tran Nguyen

1. What project are you working on?
– uncommitted flag and pre-post hook
2. What you accomplished this week?
– just come back from spring break, so minor bugs fix from the above features.
3. Links to anything you’ve done this week:
https://reviews.reviewboard.org/r/6879/
https://reviews.reviewboard.org/r/7066/

4. What you plan to do next week:
Finishing up the left over comments for these two features and move on to something new.

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

6. Any other questions?
No.


Zhiyu

1. What project are you working on?
Add per-user Default reviewer setting.

2. What you accomplished this week:
//Read the code of the api-token function ui and web-api.
Add a simple list (ui) for my new feature.
Try to provide a web api for it.

3. Links to anything you’ve done this week:
https://reviewboard.hackpad.com/Zhiyus-Notes-WLFl1aHM4fE
It will be updated tomorrow.
The diff which has not uploaded yet will be uploaded to reviewboard after my finishing it in general。

4. What you plan to do next week:
To finish it.

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

6. Any other questions?
No‍


Stanley Yeo

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

2. What you accomplished this week:
update RRs
understand local_site
understand signal

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

4. What you plan to do next week:
Implement signal handlers, and test.

5. What, if anything, is blocking you from making progress?
I’ve been busy due to midterm exams 😦

6. Any other questions?
No