Re: Issue 2899 in reviewboard: Documentation does not mention the requirement to upload a full diff each time
Comment #2 on issue 2899 by ultraran...@gmail.com: Documentation does not mention the requirement to upload a full diff each time http://code.google.com/p/reviewboard/issues/detail?id=2899 Has there been any progress on this? We've found that developers unfamiliar with ReviewBoard *always* get this wrong and upload partial diffs, which doesn't work. I'm a little surprised that there's still no official documentation to explain how to use ReviewBoard correctly. -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues. For more options, visit https://groups.google.com/groups/opt_out.
Re: Issue 2899 in reviewboard: Documentation does not mention the requirement to upload a full diff each time
Updates: Status: Fixed Comment #3 on issue 2899 by trowb...@gmail.com: Documentation does not mention the requirement to upload a full diff each time http://code.google.com/p/reviewboard/issues/detail?id=2899 I've gone in and added a note to the docs in the master branch (22f108d). Things like this can take a while because up until August of this year, Review Board has been entirely done in our spare time. We're now full time on the project, but there's a ton of work to be done on all aspects of the product (in addition to trying to build a sustainable business). -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues. For more options, visit https://groups.google.com/groups/opt_out.
Re: Issue 2899 in reviewboard: Documentation does not mention the requirement to upload a full diff each time
Updates: Labels: Component-Docs Comment #1 on issue 2899 by trowb...@gmail.com: Documentation does not mention the requirement to upload a full diff each time http://code.google.com/p/reviewboard/issues/detail?id=2899 (No comment was entered for this change.) -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues. For more options, visit https://groups.google.com/groups/opt_out.
Issue 2899 in reviewboard: Documentation does not mention the requirement to upload a full diff each time
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2899 by ultraran...@gmail.com: Documentation does not mention the requirement to upload a full diff each time http://code.google.com/p/reviewboard/issues/detail?id=2899 We run ReviewBoard 1.6.3. When submitting a diff manually (without using the post-review tool), users don't understand they need to submit complete diffs every time. Instead they may wrongly submit several partial diffs as they make changes, meaning reviewers cannot see a complete end-to-end diff, and ReviewBoard does not work as intended. This causes confusion and affects the reputation of ReviewBoard itself. People will stop using it if they can't get it to work correctly. There is a stop-gap solution that could make things easier for users immediately: The ReviewBoard documentation must explain how it's necessary to submit a full diff each time and not a partial diff. This explanation could go on the Creating Review Requests page: http://www.reviewboard.org/docs/manual/1.7/users/review-requests/creating/ If the correct documentation is in place, local administrators and advocates of ReviewBoard could direct other users to it, ensuring its correct usage. See also discussion at https://groups.google.com/forum/?fromgroups=#!starred/reviewboard/RosZjgO7FBo -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues?hl=en. For more options, visit https://groups.google.com/groups/opt_out.
