Merge pull request #454 from iBug/feature-1

Hide post feedbacks in review queues
Charcoal-SE · Jun 11, 2018 · 07895b0 · 07895b0
2 parents 7ac9f27 + 66ac3f7
commit 07895b0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/app/views/posts/_post.html.erb b/app/views/posts/_post.html.erb
@@ -1,6 +1,7 @@
 <% wrap_in_tr = true if wrap_in_tr.nil? %>
 <% expand_post = false if expand_post.nil? %>
 <% preload_post_body = expand_post || (false if preload_post_body.nil?) %>
+<% hide_feedbacks = false if hide_feedbacks.nil? %>
 <% deletion_date = false if deletion_date.nil? %>
 <% show_autoflagged = false if show_autoflagged.nil? %>
 <% show_autoflaggers = show_autoflagged if show_autoflaggers.nil? %>
@@ -20,7 +21,7 @@
       <span class="glyphicon glyphicon-flag text-danger" title="This post was autoflagged"></span>
     <% end %>
 
-    <% if not post.feedbacks.empty? %>
+    <% unless post.feedbacks.empty? or hide_feedbacks %>
       <strong>
          <% post.feedbacks.each do |feedback| %>
            <% next if (feedback.is_positive? and feedback.is_negative?) or (not feedback.is_positive? and not feedback.is_negative? and not feedback.is_naa?) %>

diff --git a/app/views/posts/_review_item.html.erb b/app/views/posts/_review_item.html.erb
@@ -1,4 +1,4 @@
 <div class="review-item">
   <%= render 'review_queues/responses', queue: queue, item: item %>
-  <%= render 'posts/post', post: item.reviewable, expand_post: true %>
-</div>
+  <%= render 'posts/post', post: item.reviewable, expand_post: true, hide_feedbacks: true %>
+</div>