Status reports for July 26

Matthew

Project: Extending Review Bot to allow tools to run on the entire repository

HackPad: https://reviewboard.hackpad.com/Review-Bot-access-to-entire-repository-QUHGKxc3M3R

  • What you accomplished this week.
    • I added a field to the ReviewBot tools for if the working_directory is required. This is configurable depending on the Tool base class used. The admin panel of the extension displays the new field.
    • Added a cached_property decorator to the project.
    • No longer copy the .git folder for a project. As a result, I changed how I apply a patch to a repository.
    • Added the Repository_Clone object into Review object and instantiate it if the tool requires the entire repository.
    • Added support to comment on a file based on it’s path. Only changed files can be commented on.
    • Created Tool to test working with the entire repository.
    • I reviewed Salam’s code: https://reviews.reviewboard.org/r/6106/
  • What you plan to do next week.
    • I will make changes according to review comments.
    • I also need to talk with Steven about steps moving forward, as this is the MVP.
  • What, if anything, is blocking you from making progress?
    • Review request for my new code.
  • Any other questions
    • No

Raheman

Project name: Per User File Storage/DnD file attachments

HackPad: https://reviewboard.hackpad.com/DnD-functionality-for-commentreview-images.-GkNpfAx8O2k

  • What you accomplished this week.
    • Started to reconsider the necessity of model/api changes.
    •  Developed a better understanding of how file attachments interact with the
      existing webapi resource.
  • What you plan to do next week.
    •  Modify my front end code to interact with the existing FileAttachment resource.
      Have a finished (but untested product) by the end of the week.
  • What, if anything, is blocking you from making progress?
    • Nothing
  • Any other questions
    •  Upon dropping should the final link be displayed or is it
      ok to use a placeholder instead?

     

Peter

Project: TogetherJS Extension

HackPadhttps://reviewboard.hackpad.com/TogetherJS-Stuff-XkauVIgVsF5

  • What you plan to do next week.
    • Whatever is left from three items above
  • What, if anything, is blocking you from making progress?
    • I’m having issues with pep8. I’m trying to fix this: https://reviews.reviewboard.org/r/6102/#comment16064 but every time I indent or unindent ReviewBot switches between being over indented and under indented.
    • When I run pep8 locally, it does not return the above correction.
  • Any other questions
    • No

Salam

Project name: Updating Checklist Extension

HackPad: https://reviewboard.hackpad.com/Updating-Checklist-Extension-rx6nO0sfA1z

  • What you accomplished this week.
    • Defined URLHook, to attache a view for the template feature
    • Connected the templates resource api to the model
    • Defined a view.py that returned the default template.
    • Worked on html and css of the options page
  • What you plan to do next week.
    • finish implementing view.py

     

  • What, if anything, is blocking you from making progress?
    • Nothing
  • Any other questions
    • No

Volodymyr

Project name: Reviewing attachments

HackPad: https://reviewboard.hackpad.com/Reviewing-attachments-tbfh4oiwTHB

  • What you accomplished this week.
    • Fixed the new thumbnail look and functionality – it works now (review to be updated)
    • A generic review for files without a tyoe-specific handler
    • Iteration on https://reviews.reviewboard.org/r/5911/
  • What you plan to do next week.

     

  • What, if anything, is blocking you from making progress?
    • Nothing
  • Any other questions
    • No
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