Merge branch '25093-hide-new-issue-btn-non-loggedin-user' into 'master'

hides new issue btn for not loggedin user ## What does this MR do? This MR closes #25093 #25180 ## Are there points in the code the reviewer needs to double check? NR ## Why was this MR needed? Prevent issue creation for no logged in user and some confusion related to ## Screenshots (if relevant) ## Does this MR meet the acceptance criteria? - [x] [Changelog entry](https://docs.gitlab.com/ce/development/changelog.html) added - Tests - [ ] All builds are passing - [x] Branch has no merge conflicts with `master` (if it does - rebase it please) - [ ] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) ## What are the relevant issue numbers? Closes #25093, #25180 See merge request !8175

Merge branch '25093-hide-new-issue-btn-non-loggedin-user' into 'master'
2ccfeefe · Fatih Acet · 4909d77d · 92af7bfc · 2ccfeefe · 2ccfeefe
Commit 2ccfeefe authored Dec 20, 2016 by Fatih Acet
Showing with 5 additions and 1 deletion

_issues.html.haml app/views/shared/empty_states/_issues.html.haml +1 -1

25093-hide-new-issue-btn-non-loggedin-user.yml ...unreleased/25093-hide-new-issue-btn-non-loggedin-user.yml +4 -0

No files found.
--- a/app/views/shared/empty_states/_issues.html.haml
+++ b/app/views/shared/empty_states/_issues.html.haml
@@ -8,7 +8,7 @@
      = render 'shared/empty_states/icons/issues.svg'
  .col-xs-12{ class: "#{'col-sm-6' if has_button}" }
    .text-content
-      - if has_button
+      - if has_button && current_user
        %h4
          The Issue Tracker is a good place to add things that need to be improved or solved in a project!
        %p

--- a/changelogs/unreleased/25093-hide-new-issue-btn-non-loggedin-user.yml
+++ b/changelogs/unreleased/25093-hide-new-issue-btn-non-loggedin-user.yml
+---
+title: Hides new issue button for non loggedin user
+merge_request: 8175
+author: