The following discussion from !43290 should be addressed: - [ ] @mwoolf started a [discussion](https://gitlab.com/gitlab-org/gitlab/-/merge_requests/43290#note_434136751): (+4 comments) > I'm not totally convinced by this method of specifying a redirect. :thinking: `GroupsController#update` is a **very** heavily utilized controller action and this change is expanding it quite a lot and adding a nested conditional statement. > > I wonder if this logic belongs in a separate controller entirely, similar to the deploy tokens settings on the same page? Or at the least, a different action within this controller? WDYT? > > As an example, updating CI/CD settings posts to `Groups::Settings::CiCdController#update_auto_devops` via `/groups/GROUP_NAME/-/settings/ci_cd/update_auto_devops` rather than the groups controller