Author Archives: chenxi94

[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.

Advertisements