Skip to content

[RUN-AS-IF-FOSS] Removed unused HAML templates

Robert Hunt requested to merge 214811-delete-unused-haml-templates into master

What does this MR do?

While working on !40585 (merged) I realised that all the HAML templates under app/views/projects/merge_requests/diffs/ are no longer used! They aren't mentioned in the codebase (expect within each other) and the root controller was changed in the past to JSON only, which means these templates can never be reached.

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team

Related #224534 (closed)

Edited by Paul Slaughter

Merge request reports