mozreview: show review flag in review request history (bug 1197879); r?smacleod draft
authorMauro Doglio <mdoglio@mozilla.com>
Tue, 05 Apr 2016 13:54:27 +0100
changeset 8122 cf687187e8aa43e745bebd147097bba26ae5b491
parent 8121 84ca3ce51ec1bb6aa5f012913e370f72c4e05523
child 8123 05d98e90e38c46a40e56e9ea8326571f9251839c
push id830
push usermdoglio@mozilla.com
push dateTue, 17 May 2016 10:03:33 +0000
reviewerssmacleod
bugs1197879
mozreview: show review flag in review request history (bug 1197879); r?smacleod MozReview-Commit-ID: D6M2bqEta78
pylib/mozreview/mozreview/static/mozreview/js/review.js
--- a/pylib/mozreview/mozreview/static/mozreview/js/review.js
+++ b/pylib/mozreview/mozreview/static/mozreview/js/review.js
@@ -17,9 +17,39 @@
 
   // Change string of "Review" button to be a verb so people better
   // understand what clicking it does.
   $("#review-link").text("Finish Review...");
 
   if (MozReview.isParent) {
     $('#review_request_extra').prepend(MRParents.parentWarning);
   }
+
+  var reviewRequest = RB.PageManager.getPage().reviewRequest;
+  RB.apiCall({
+    type: 'GET',
+    prefix: reviewRequest.get('sitePrefix'),
+    noActivityIndicator: true,
+    url: '/api/review-requests/'+reviewRequest.get('id')+'/reviews/'
+       + '?max-results=200',
+    success: function(data) {
+      console.log(data);
+      _.forEach(data.reviews, function(item) {
+        var flag = item.extra_data['p2rb.review_flag'];
+        var flagDesc = '';
+        var reviewText = $('#review'+item.id+' .body');
+        switch(flag){
+          case ' ':
+            flagDesc = 'Review flag cleared';
+            break;
+          case 'r-':
+          case 'r+':
+          case 'r?':
+            flagDesc = 'Review flag: '+flag;
+            break;
+        }
+        $(reviewText).prepend(
+          $('<h4 class="body_top">'+flagDesc+'</h4>')
+        );
+      })
+    }
+  });
 });