UX: Slightly improved reviewable user display, plus link to admin if possible

app/assets/javascripts/discourse/templates/components/reviewable-user.hbs

@@ -1,7 +1,25 @@
 <div class='reviewable-user-info'>
-  <div class='reviewable-user-details'>
+  <div class='reviewable-user-details username'>
+    <div class='name'>{{i18n "review.user.username"}}</div>
+    <div class='value'>
+      {{#if reviewable.link_admin}}
+        <a href={{get-url (concat "/admin/users/" reviewable.user_id "/" reviewable.payload.username)}}>
           {{reviewable.payload.username}}
-    {{reviewable.payload.email}}
+        </a>
+      {{else}}
+        {{reviewable.payload.username}}
+      {{/if}}
+    </div>
+  </div>
+  {{#if reviewable.payload.name}}
+    <div class='reviewable-user-details name'>
+      <div class='name'>{{i18n "review.user.name"}}</div>
+      <div class='value'>{{reviewable.payload.name}}</div>
+    </div>
+  {{/if}}
+  <div class='reviewable-user-details email'>
+    <div class='name'>{{i18n "review.user.email"}}</div>
+    <div class='value'>{{reviewable.payload.email}}</div>
   </div>
 
   {{yield}}

app/assets/stylesheets/common/base/reviewables.scss

@@ -44,8 +44,18 @@
   }
 }
 
-.reviewable-user-details {
+.reviewable-user-info {
   margin: 0.5em 0;
+
+  .reviewable-user-details {
+    display: flex;
+    .name {
+      width: 8em;
+      font-weight: bold;
+      margin-right: 1em;
+    }
+    margin-bottom: 0.5em;
+  }
 }
 
 .no-review {

app/serializers/reviewable_user_serializer.rb

@@ -1,9 +1,15 @@
 class ReviewableUserSerializer < ReviewableSerializer
 
+  attributes :link_admin
+
   payload_attributes(
     :username,
     :email,
     :name
   )
 
+  def link_admin
+    scope.is_staff? && object.target.present?
+  end
+
 end

app/services/user_destroyer.rb

@@ -29,7 +29,7 @@ class UserDestroyer
       Reviewable.where(created_by_id: user.id).delete_all
 
       if reviewable = Reviewable.find_by(target: user)
-        reviewable.perform(@actor, :reject, skip_delete: true)
+        reviewable.perform(@actor, :reject, skip_delete: true) rescue Reviewable::InvalidAction
       end
 
       if opts[:delete_posts]

config/locales/client.en.yml

@@ -381,6 +381,11 @@ en:
       filtered_user: "User:"
       show_all_topics: "show all topics"
       deleted_user: "(deleted user)"
+      user:
+        username: "Username"
+        email: "Email"
+        name: "Name"
+
       user_percentage:
         summary:
           one: "{{agreed}}, {{disagreed}}, {{ignored}} ({{count}} total flag)"