Skip to content
Snippets Groups Projects

Removal of more backend Visual Review code

Merged drew stachon requested to merge 387751-deprecation-visual-review-be into master

What does this MR do and why?

This removes a bunch of dead code that's no longer used as of Remove visual review tools (!149586 - merged)

There was more removal than I thought, so we removed interfaces and anything user-facing in the first MR, and are sending follow-ups to do more removal behind the scenes.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

How to set up and validate locally

Just run the test suite. All the changes in here are removals, so there's no additive thing to go look at. If everything else still works, we're good.

Related to #387751 (closed)

Edited by drew stachon

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • drew stachon
  • drew stachon marked this merge request as draft

    marked this merge request as draft

  • drew stachon mentioned in merge request !150175 (merged)

    mentioned in merge request !150175 (merged)

  • drew stachon removed milestone %17.0

    removed milestone %17.0

  • Author Maintainer

    Removing the milestone and marking this as a Draft, to be superseded by a smaller backend removal for %17.0: Remove all interfaces to Visual Review feature (!150175 - merged)

    frontend removal in Remove visual review tools (!149586 - merged)

    We can continue this effort as a dead-code cleanup after those official removal MRs ship in %17.0.

  • Author Maintainer

    Making myself a note to remote this layout file in this MR

  • Payton Burdette deleted the 387751-deprecation-visual-review-fe branch. This merge request now targets the master branch

    deleted the 387751-deprecation-visual-review-fe branch. This merge request now targets the master branch

  • drew stachon added 1 commit

    added 1 commit

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading