diff options
author | Fatih Acet <acetfatih@gmail.com> | 2016-07-01 20:09:27 +0000 |
---|---|---|
committer | Robert Speicher <rspeicher@gmail.com> | 2016-07-06 12:48:51 -0400 |
commit | b7a6252497547f1187f17918f0b49bcc6e64eccb (patch) | |
tree | 5bba785a70d58cf1557a93054f1beeeb8ce1cb46 | |
parent | 1b85f5572a0cdd8326bf1453f5afe0bb981a3609 (diff) | |
download | gitlab-ce-b7a6252497547f1187f17918f0b49bcc6e64eccb.tar.gz |
Merge branch 'renable-import-button-when-namespace-already-been-taken' into 'master'
Renable import button when import process fail due to the namespace already been taken
Closes #19435
## Screenshots (if relevant)
Before:
![1](/uploads/e8de1b326e0751891f667630a7685f6a/1.png)<br/><br/>
After:
![2](/uploads/566f1fd5442c28232350689fce8eae76/2.png)
See merge request !5053
(cherry picked from commit d6efef0f2aeb982dcd64c63db1ec8b4946f77a31)
-rw-r--r-- | CHANGELOG | 1 | ||||
-rw-r--r-- | app/assets/javascripts/importer_status.js.coffee | 11 | ||||
-rw-r--r-- | app/views/import/base/create.js.haml | 5 |
3 files changed, 12 insertions, 5 deletions
diff --git a/CHANGELOG b/CHANGELOG index 2326ce5aac5..bfbc90274aa 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -1,6 +1,7 @@ Please view this file on the master branch, on stable branches it's out of date. v 8.9.5 + - Re-enable import button when import process fails due to namespace already being taken. !5053 - Fix snippets comments not displayed. !5045 - Fix emoji paths in relative root configurations. !5027 - Fix issues importing events in Import/Export. !4987 diff --git a/app/assets/javascripts/importer_status.js.coffee b/app/assets/javascripts/importer_status.js.coffee index b0edc895649..eb046eb2eff 100644 --- a/app/assets/javascripts/importer_status.js.coffee +++ b/app/assets/javascripts/importer_status.js.coffee @@ -7,13 +7,16 @@ class @ImporterStatus $('.js-add-to-import') .off 'click' .on 'click', (e) => - new_namespace = null $btn = $(e.currentTarget) $tr = $btn.closest('tr') + $target_field = $tr.find('.import-target') + $namespace_input = $target_field.find('input') id = $tr.attr('id').replace('repo_', '') - if $tr.find('.import-target input').length > 0 - new_namespace = $tr.find('.import-target input').prop('value') - $tr.find('.import-target').empty().append("#{new_namespace} / #{$tr.find('.import-target').data('project_name')}") + new_namespace = null + + if $namespace_input.length > 0 + new_namespace = $namespace_input.prop('value') + $target_field.empty().append("#{new_namespace}/#{$target_field.data('project_name')}") $btn .disable() diff --git a/app/views/import/base/create.js.haml b/app/views/import/base/create.js.haml index dfebf7768d9..804ad88468f 100644 --- a/app/views/import/base/create.js.haml +++ b/app/views/import/base/create.js.haml @@ -1,6 +1,8 @@ - if @already_been_taken :plain - target_field = $("tr#repo_#{@repo_id} .import-target") + tr = $("tr#repo_#{@repo_id}") + target_field = tr.find(".import-target") + import_button = tr.find(".btn-import") origin_target = target_field.text() project_name = "#{@project_name}" origin_namespace = "#{@target_namespace}" @@ -10,6 +12,7 @@ target_field.append("/" + project_name) target_field.data("project_name", project_name) target_field.find('input').prop("value", origin_namespace) + import_button.enable().removeClass('is-loading') - elsif @access_denied :plain job = $("tr#repo_#{@repo_id}") |