Status Reports (Nov 16th 2012)

8 views
Skip to first unread message

Tina Yang

unread,
Nov 16, 2012, 4:48:29 PM11/16/12
to reviewbo...@googlegroups.com
Hi everyone, I would like to kick off the status reports for this week :

Tina Yang

Currently: 
- Working on resolving a CSS issue on Chrome only for inline binary file UI and cleaning up code for code review.

Roadblocks:
- None

Next:
- Upload binary files via post-review (Git)

Questions:
- None

Jesus Zambrano

unread,
Nov 16, 2012, 6:24:35 PM11/16/12
to reviewbo...@googlegroups.com
Jesus Zambrano

Currently:
- Changing inlineEditor so that disable editable fields when a review request is discarded.
-Fixed the issue of more than one pencil icon on the discard-banner and submitted-banner

Roadblocks:
- Nothing major, just somewhat confused as to how the inlineEditor works.

Next:
- Add a field to RB.ReviewRequest to check if a review request is public or a draft. This would help display the appropriate banner(or no banner) when a submitted review request is re-opened.

Questions:
- None

Allyshia Sewdat

unread,
Nov 16, 2012, 8:23:52 PM11/16/12
to reviewbo...@googlegroups.com
Allyshia Sewdat

Currently:
- Updated WIP review request for JSLint plugin with 'final' submission pending any change requests. For now this looks like the final state in which I will leave that particular project for the term, not counting potential polishing later on. The WIP status could be removed as soon as some additional conclusive testing is completed to confirm performance quality, even though functionality has been proven.
- Got some feedback on mockups for the manual trigger of static analysis.
- Looking into the codebase to figure out how to hook up the new manual trigger functionality.

Roadblocks:
- None for now

Next:
- Continue the exploration of the codebase to come up with a sound design for the manual trigger, taking into account the tips given to me by mentors.
- Will continue to accept feedback on the WIP review request for JSLint plugin and make adjustments.

Questions:
- Any immediate tips or gotchas that come to mind regarding implementing a new navigation bar option / implementing the ReviewBot manual trigger extension would be appreciated. Otherwise I'll keep pinging people for guidance.

Aamir Mansoor

unread,
Nov 17, 2012, 9:05:05 AM11/17/12
to reviewbo...@googlegroups.com
Aamir Mansoor

Currently:
- Took Markdown Pluggable UI out of WIP. It can be reviewed at r/3434
- Looking into loading pluggable UI in a lightbox rather than redirecting to a new page

Roadblocks:
- None

Next:
- Continue working on rendering pluggable UI in lightbox

Questions:
- None

Aamir Mansoor

Sampson Chen

unread,
Nov 17, 2012, 11:16:31 AM11/17/12
to reviewbo...@googlegroups.com
Sampson Chen

Currently:
- Working on ReviewUI integration with Extensions. Making progress, should have a WIP up later today.

Roadblocks:
- None, just been crazy busy lately

Next:
- Add in changes for memcaching for thumbnails.

Questions:
- None

Wesley Ellis

unread,
Nov 17, 2012, 12:25:06 PM11/17/12
to reviewbo...@googlegroups.com
Wesley Ellis

Currently:
- Writing the settings panel for buildbot

Roadblocks:
- Can't get the buildbot try command to accept diffs
- Cant' get buildbot try to connect via ssh

Next:
- fix roadblocks
-

Questions:
- None
On Sat, Nov 17, 2012 at 9:05 AM, Aamir Mansoor <m...@aam1r.com> wrote:
Aamir Mansoor

Currently:
- Took Markdown Pluggable UI out of WIP. It can be reviewed at r/3434
- Looking into loading pluggable UI in a lightbox rather than redirecting to a new page

Roadblocks:
- None

Next:
- Continue working on rendering pluggable UI in lightbox

Questions:
- None

Aamir Mansoor
On Fri, Nov 16, 2012 at 8:23 PM, Allyshia Sewdat <ally...@gmail.com> wrote:

John Sintal

unread,
Nov 17, 2012, 1:06:22 PM11/17/12
to reviewbo...@googlegroups.com
John Sintal

Currently:
- Finish rb patch and addressing David's issues

Roadblocks:
- Can't eliminate certain options due to complaints in legacy code
- Trying to figure out pX logic, but the base dir given by repository_info is always "/"
- Figuring out how to setup SVN

Next:
- fix roadblocks

Questions:
- I'll be posting a new patch very soon with regards to my road blocks. If anyone can help me eliminate the superfluous options or give pointers on how to deal with the legacy code that expect these options to exist when obtaining a SCM.
- Do we have a testing repo for SVN?

Karl L.

unread,
Nov 17, 2012, 2:01:08 PM11/17/12
to reviewbo...@googlegroups.com
Karl Leuschen

Currently:
- Monitoring my active review requests and making changes as suggested
- Submitted blog post for the week
- Looking into Bitbucket API

Roadblocks:
- Need someone to take a look at r/3488. It's not stopping me from having work, but I'll need some help on it in order to finish my project's core feature.

Next:
- Work on finishing WIP patch (r/3488)
- Start getting connected with Bitbucket
- Look at more open review requests

Questions:
- None.

Michelle Chuang

unread,
Nov 17, 2012, 3:07:53 PM11/17/12
to reviewbo...@googlegroups.com
Michelle Chuang

Currently:
- Addressed code review comments for r/3458
- Working on creating a view with multiple image diffing options
- Working on image diffing through a "swipe view"

Roadblocks:
- None

Next:
- Continue to work on different image diffing views

Questions:
- None
Reply all
Reply to author
Forward
0 new messages