Merge branch 'add-editable-assign-yourself' into 'multiple_assignees_review'

Show assign yourself when assignees is editable

See merge request !1763

app/assets/javascripts/sidebar/components/assignees/assignees.js

@@ -16,6 +16,10 @@ export default {
       type: Array,
       required: true,
     },
+    editable: {
+      type: Boolean,
+      required: true,
+    },
   },
   computed: {
     firstUser() {
@@ -142,14 +146,17 @@ export default {
       <div class="value hide-collapsed">
         <template v-if="hasNoUsers">
           <span class="assign-yourself no-value">
-            No assignee -
-            <button
-              type="button"
-              class="btn-link"
-              @click="assignSelf"
-            >
-              assign yourself
-            </button>
+            No assignee
+            <template v-if="editable">
+             -
+              <button
+                type="button"
+                class="btn-link"
+                @click="assignSelf"
+              >
+                assign yourself
+              </button>
+            </template>
           </span>
         </template>
         <template v-else-if="hasOneUser">

app/assets/javascripts/sidebar/components/assignees/sidebar_assignees.js

@@ -76,6 +76,7 @@ export default {
         class="value"
         :root-path="store.rootPath"
         :users="store.assignees"
+        :editable="store.editable"
         @assign-self="assignSelf"
       />
     </div>

app/views/projects/boards/components/sidebar/_assignee.html.haml

@@ -5,6 +5,7 @@
       ":editable" => can?(current_user, :admin_issue, @project) }
     %assignees.value{ "root-path" => "#{root_url}",
       ":users" => "issue.assignees",
+      ":editable" => can?(current_user, :admin_issue, @project),
       "@assign-self" => "assignSelf" }
 
   - if can?(current_user, :admin_issue, @project)