Skip to content

Resolve "Hide or disable security reports feedback actions when not authorized"

Filipa Lacerda requested to merge 6078-add-permissions-checks-dismiss-issue into master

What does this MR do?

  • Adds permissions checks for dismiss issue and create issue buttons in security reports

Screenshots (if relevant)

no permissions can create issue can dismiss
Screen_Shot_2018-06-26_at_11.42.52 Screen_Shot_2018-06-19_at_10.57.20 Screen_Shot_2018-06-19_at_10.57.29

Does this MR meet the acceptance criteria?

  • Changelog entry added, if necessary
  • Tests added for this feature/bug
  • Conform by the code review guidelines
    • Has been reviewed by a Frontend maintainer
    • Has been reviewed by a Backend maintainer
  • EE specific content should be in the top level /ee folder
  • Conform by the merge request performance guides
  • Conform by the style guides
  • If you have multiple commits, please combine them into a few logically organized commits by squashing them
  • Internationalization required/considered

What are the relevant issue numbers?

Closes #6078 (closed)

Edited by Phil Hughes

Merge request reports