Is there a print function in review board?
Hi, Just wonder is there a print function in review board? In case, a face to face review is needed? Thanks, -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en
Re: Is there a print function in review board?
Hi Nelson, There's just the standard browser print capabilities. We don't do any special formatting. Christian On Friday, September 30, 2011, Nelson Jones huawen@gmail.com wrote: Hi, Just wonder is there a print function in review board? In case, a face to face review is needed? Thanks, -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en -- -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en
Re: Is there a print function in review board?
I see. Thanks Christian! On Sep 30, 12:00 pm, Christian Hammond chip...@chipx86.com wrote: Hi Nelson, There's just the standard browser print capabilities. We don't do any special formatting. Christian On Friday, September 30, 2011, Nelson Jones huawen@gmail.com wrote: Hi, Just wonder is there a print function in review board? In case, a face to face review is needed? Thanks, -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know athttp://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en -- -- Christian Hammond - chip...@chipx86.com Review Board -http://www.reviewboard.org VMware, Inc. -http://www.vmware.com -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en
Re: Is there a print function in review board?
One option is to print the email sent from reviewboard. If you want to print the diffs you are likely to find sizing issues with the web page approach with the wide side-by-side diff view. We've modified our RB server to include the diff in the email that is sent out for review (we've not yet worked out how to NOT include the diffs in replies/comments). That can help a little bit but the main advantage we've found is using the browser to see the diffs and then immediately expand code and add comments. You could do this with face-to-face reviews if the computer is used instead of print. Chris On 9/30/2011 9:06 AM, Nelson Jones wrote: I see. Thanks Christian! On Sep 30, 12:00 pm, Christian Hammondchip...@chipx86.com wrote: Hi Nelson, There's just the standard browser print capabilities. We don't do any special formatting. Christian On Friday, September 30, 2011, Nelson Joneshuawen@gmail.com wrote: Hi, Just wonder is there a print function in review board? In case, a face to face review is needed? Thanks, -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en
Re: Is there a print function in review board?
On Fri, 2011-09-30 at 11:32 -0700, Chris Clark wrote: One option is to print the email sent from reviewboard. If you want to print the diffs you are likely to find sizing issues with the web page approach with the wide side-by-side diff view. We've modified our RB server to include the diff in the email that is sent out for review (we've not yet worked out how to NOT include the diffs in replies/comments). That can help a little bit but the main advantage we've found is using the browser to see the diffs and then immediately expand code and add comments. You could do this with face-to-face reviews if the computer is used instead of print. Does it send the diff in the email contents or as an attachment? If the latter, could you share your patch with the rest of the class? :) -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en
Re: Is there a print function in review board?
On 9/30/2011 11:34 AM, Stephen Gallagher wrote: On Fri, 2011-09-30 at 11:32 -0700, Chris Clark wrote: One option is to print the email sent from reviewboard. If you want to print the diffs you are likely to find sizing issues with the web page approach with the wide side-by-side diff view. We've modified our RB server to include the diff in the email that is sent out for review (we've not yet worked out how to NOT include the diffs in replies/comments). That can help a little bit but the main advantage we've found is using the browser to see the diffs and then immediately expand code and add comments. You could do this with face-to-face reviews if the computer is used instead of print. Does it send the diff in the email contents or as an attachment? It depends :-) It is semi-configurable, i.e. there is code that has a parameter but it is hard coded at the moment. We default to inline (this matches our old review standards that predated re) but switch to an attachment if we have no idea what the encoding of the diff is. We need to add a size filter, e.g. we had a monster review the other day and what should have happened is the diff should have been compressed and attached. If the latter, could you share your patch with the rest of the class? :) Right now the code is in a private git repo :-( This is an excellent reminder that we should try and make this public even if no one will want all of it. Right now what we have is NOT ready for prime time As a taster, here is the code, it is not pep8 compliant and it isn't a diff (we're on an older version), but it should allow you to make a similar (more robust!) change: in reviewboard/templates/notifications/review_request_email.html: {% if raw_diff %} table width=100% bgcolor=#ff cellspacing=0 cellpadding=10 style=border: 1px solid #b8b5a0 tr td pre style=margin: 0; padding: 0; {{precss}}{{ raw_diff }}/pre /td /tr /table {% endif %} --- this is the inline diff option In /reviewboard/notifications/email.py: context['MEDIA_SERIAL'] = settings.MEDIA_SERIAL # should really (add to) repository and add include raw diffs option # for now check scm type or always do it! include_raw_diff=True #include_raw_diff=False ## DEBUG FIXME REMOVE! ## NOTE some of our mailing list do not retain (diff) attachments. attachment_message='Diffs attached' emaildiffs_inline=True ## if False make an attachment context['raw_diff']='' #context['raw_diff']='debug raw diff FIXME REMOVE' if include_raw_diff: ## Other option, instead of a flag include_raw_diff could lazy evaluate ## http://pyds.muensterland.org/ has a nice short one (only needs minor cleanup, and I already have a cleaned up copy) diffset = review_request.diffset_history.diffsets.latest() raw_diff = review_request.repository.get_scmtool().get_parser('').raw_diff(diffset) if emaildiffs_inline and isinstance(raw_diff, str): try: _ = raw_diff.decode('us-ascii') # should check repository encoding type.. except UnicodeDecodeError: # non-ASCII data in diff, email is (probably) utf8 and # so will not display inline. Switch to an attachment emaildiffs_inline=False ## if False make an attachment attachment_message='Diffs contain non-ASCII characters/bytes, unable to determine encoding. See attachment.' if emaildiffs_inline: context['raw_diff']=raw_diff else: # attach context['raw_diff']=attachment_message message = SpiffyEmailMessage(subject.strip(), text_body, html_body, from_email, list(to_field), list(cc_field), in_reply_to, headers) if include_raw_diff and not emaildiffs_inline: message.attach('diff.diff', raw_diff, 'text/x-diff') -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en