Commit 63b4c19d authored by Douwe Maan's avatar Douwe Maan

Merge branch '5194-fix-github-ci-cd-github-auth-missing-query-param' into 'master'

Add `?ci_cd_only=true` query parameter to CI/CD GitHub status link

Closes #5194

See merge request gitlab-org/gitlab-ee!4914
parents 91e2fdf5 365b1280
......@@ -10,7 +10,7 @@
%p
= import_github_authorize_message
= link_to _('List your GitHub repositories'), status_import_github_path, class: 'btn btn-success'
= link_to _('List your GitHub repositories'), status_import_github_path(ci_cd_only: params[:ci_cd_only]), class: 'btn btn-success'
%hr
......
......@@ -139,6 +139,12 @@ feature 'New project' do
expect(created_project.project_feature).not_to be_issues_enabled
end
it 'new GitHub CI/CD project page has link to status page with ?ci_cd_only=true param' do
visit new_import_github_path(ci_cd_only: true)
expect(page).to have_link('List your GitHub repositories', href: status_import_github_path(ci_cd_only: true))
end
it 'stays on GitHub import page after access token failure' do
visit new_project_path
find('#ci-cd-project-tab').click
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment