Fix the misleading string in pop up "RBTool Setup" message in repository admin
Review Request #6788 — Created Jan. 16, 2015 and submitted
In the repository list in the Review Board admin site, we have an "RBTools Setup" link that pops up instructions for what to put in a .reviewboardrc file. One instruction was misleading (telling the user to work on root directory) in cases like SVN where the user won't be checking out the "repository root", but rather a subdirectory
The misleading string is produced by reviewboard/templates/admin/scmtools/repository/rbtools_setup.html I just changed some static string in the html file.
This change fixes the text to suggest users commit this file to the root of their working copy.
Show better string on Chrome after modification
Description | From | Last Updated |
---|---|---|
Please wrap this to 80 columns. |
david |
Branch: |
|
||||
---|---|---|---|---|---|
Commit: |
|
||||
Diff: |
Revision 2 (+1 -1) |
-
Tool: PEP8 Style Checker Ignored Files: reviewboard/templates/admin/scmtools/repository/rbtools_setup.html Tool: Pyflakes Ignored Files: reviewboard/templates/admin/scmtools/repository/rbtools_setup.html
-
Your review request description needs work. It should explain what the bug is and how your change fixes it.
-
-
reviewboard/templates/admin/scmtools/repository/rbtools_setup.html (Diff revision 2) Please wrap this to 80 columns.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 3 (+2 -2) |
-
Tool: Pyflakes Ignored Files: reviewboard/templates/admin/scmtools/repository/rbtools_setup.html Tool: PEP8 Style Checker Ignored Files: reviewboard/templates/admin/scmtools/repository/rbtools_setup.html
-
Your summary/description is improved, but still not very useful. For example, looking through git history, I would wonder "what string?" and "what pop up message"? You should also endeavor to use correct capitalization and punctuation.
Here's an example of what we might expect for this bug fix:
Fix misleading "RBTools Setup" instructions in the repository admin. In the repository list in the Review Board admin site, we have an "RBTools Setup" link that pops up instructions for what to put in a .reviewboardrc file. These instructions were misleading in cases like SVN where the user won't be checking out the "repository root", but rather a subdirectory. This change fixes the text to suggest users commit this file to the root of their working copy.
Summary: |
|
||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Description: |
|
Description: |
|
---|