Moves localized string to javascript, in the prompt to discard a draft review
Review Request #6346 — Created Sept. 20, 2014 and submitted
Making the last few changes on code that fixed an issue, where there was no prompt after pressing the Discard Review button on draft reviews.
This moves embedded localized strings to javascript from of HTML.
Tested Visually (in both Chrome and Firefox)
Both tested with and without clicking 'ok' on the draft review under a comment or issue.
. Clicking the Discard Review button, then clicking cancel closes the prompt.
. Clicking the Discard Review button, then clicking discard exits the review and gets you back to the review request.
| Description | From | Last Updated |
|---|---|---|
|
There's no trailing comma here, so this will actaully break. Make sure you test this, and have Testing Done reflect … |
 chipx86
|
- Change Summary:
-
rebased on origin/master branch
- Commit:
-
ee08d4048e4464ad4eb11b64b7e3908094a027d7df5af813ff3df8f53eca0d9d5e52d708840a41a5
-
Tool: PEP8 Style Checker Ignored Files: reviewboard/static/rb/js/views/reviewDialogView.js AUTHORS Tool: Pyflakes Ignored Files: reviewboard/static/rb/js/views/reviewDialogView.js AUTHORS
- Change Summary:
-
rebases on the new master
- Commit:
-
df5af813ff3df8f53eca0d9d5e52d708840a41a56e89add5eda55e1946e7bc477d1290ef1549fdb4
-
Tool: PEP8 Style Checker Ignored Files: reviewboard/static/rb/js/views/reviewDialogView.js AUTHORS Tool: Pyflakes Ignored Files: reviewboard/static/rb/js/views/reviewDialogView.js AUTHORS
- Summary:
-
fixing Issue 3300:Add a confirmation when discarding a pending reviewIssue 3300 last few changes
- Description:
-
~ Issue 3300 - last changes
~ Making the last few changes on issue 3300.
~ embedding localized string in javascript instead of HTML
~ Embedding localized string in javascript instead of HTML.
- Testing Done:
-
Same as here: https://reviews.reviewboard.org/r/6341/
-
Make sure every single summary, description, and testing done can be read without knowledge of other changes or bug numbers. "Issue XXX" shoudn't be in the summary, just in the bugs field, and I should be able to get a sense of the change from seeing just the summary, with complete details by reading the description.
- Change Summary:
-
Adding detail to the 'testing done' field
- Testing Done:
-
~ Same as here: https://reviews.reviewboard.org/r/6341/
~ Tested Visually in both Chrome and Firefox
+ + Both tested with and without clicking 'ok' on the review under a comment or issue.
+ + . Clicking the Discard Review button, then clicking cancel closes the prompt.
+ . Clicking the Discard Review button, then clicking discard exits the review and gets you back to the review request.
- Change Summary:
-
added in missing comma on line 773
- Commit:
-
6e89add5eda55e1946e7bc477d1290ef1549fdb470e20e53439825b84dd3a3a5b155808ec7f73173
-
Tool: Pyflakes Ignored Files: reviewboard/static/rb/js/views/reviewDialogView.js AUTHORS Tool: PEP8 Style Checker Ignored Files: reviewboard/static/rb/js/views/reviewDialogView.js AUTHORS
-
Looks good! Just one thing - can you modify your summary and description such that someone could read and understand it without reading up on the bug (issue 3300) you're referencing?
- Change Summary:
-
Updated the summary, description, and testing done.
- Summary:
-
Issue 3300 last few changesMoves localized string to javascript, in the prompt to discard a draft review
- Description:
-
~ Making the last few changes on code that fixes issue 3300.
~ Making the last few changes on code that fixed an issue, where there was no prompt after pressing the Discard Review button on draft reviews.
~ Embedding localized string in javascript instead of HTML.
~ This moves embedded localized strings to javascript from of HTML.
- Testing Done:
-
~ Tested Visually in both Chrome and Firefox
~ Tested Visually (in both Chrome and Firefox)
~ Both tested with and without clicking 'ok' on the review under a comment or issue.
~ Both tested with and without clicking 'ok' on the draft review under a comment or issue.
. Clicking the Discard Review button, then clicking cancel closes the prompt.
. Clicking the Discard Review button, then clicking discard exits the review and gets you back to the review request.
Tool: Pyflakes Ignored Files: reviewboard/static/rb/js/views/reviewDialogView.js AUTHORS Tool: PEP8 Style Checker Ignored Files: reviewboard/static/rb/js/views/reviewDialogView.js AUTHORS