Skip to content

Remove obsolete view on environment button in MR diffs

Bala Kumar requested to merge rm-environments-in-mr-diffs into master

What does this MR do and why?

Removes view on environment button from the MR diffs header. We discussed and agreed to remove the button as the feature is already broken and we have query timeouts (sentry issue link) happening on the MR diffs even though the feature is broken.

View on environment feature is also available in the MR widget which is the actively maintained and developed area these days.

Screenshots or screen recordings

screen_shot_view_on_environment_rm

How to set up and validate locally

  1. Create an MR in local repository (can be http://localhost:3000/flightjs/Flight), with the patch file add-new-file.patch
  2. View the MR diff changes as shown in the screencast and view on environment button should not be visible on the MR diffs header.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #349769 (closed)

Edited by Bala Kumar

Merge request reports