Revert "Fix: No default selection of destination namespace when importing with GH, BB ad Gitea"
Purpose of revert
Sentry is reporting the following unhandled error, and has reported about a dozen incidents in the last 12 hours.
We're not yet able to recreate the error in the browser, as the frontend import validation is working correctly (a user is not able to import without selecting a target namespace). However, it could be that there is a path to import that wasn't covered by the original MR. According to the logs, this was a web request.
Reverting in order to investigate
https://new-sentry.gitlab.net/organizations/gitlab/issues/1602627/?project=3
ArgumentErrorTarget namespace is required (ArgumentError) raise ArgumentError, s_('GithubImport|Target namespace is required') if params[:target_namespace].blank? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^Checklist
-
Create an issue to reinstate the merge request and assign it to the author of the reverted merge request. -
If the revert is to resolve a broken 'master' incident, please read through the Responsibilities of the Broken masterresolution DRI. -
If the revert involves a database migration, please read through Deleting existing migrations. -
Add the appropriate labels before the MR is created. We can skip CI/CD jobs only if the labels are added before the CI/CD pipeline is created.
Milestone info
-
I am reverting something in the current milestone. No changelog is needed, and I've added a ~"regression:*"label. -
I am reverting something in a different milestone. A changelog is needed, and I've removed the ~"regression:*"label.
Related issues and merge requests
MR - !193193 (merged)
Issue - #534663
Edited by Oiza Baiye