Show error message on jira import fail
parent
ff7ac8498a
commit
48d4c53810
|
@ -28,9 +28,10 @@ class ImportProjectController
|
|||
'$routeParams',
|
||||
'$tgNavUrls',
|
||||
'$tgConfig',
|
||||
'$tgConfirm',
|
||||
]
|
||||
|
||||
constructor: (@trelloService, @jiraService, @githubService, @asanaService, @location, @window, @routeParams, @tgNavUrls, @config) ->
|
||||
constructor: (@trelloService, @jiraService, @githubService, @asanaService, @location, @window, @routeParams, @tgNavUrls, @config, @confirm) ->
|
||||
|
||||
start: ->
|
||||
@.token = null
|
||||
|
@ -88,7 +89,9 @@ class ImportProjectController
|
|||
@window.open(url, "_self")
|
||||
else if from == "jira"
|
||||
@jiraService.getAuthUrl(@.jiraUrl).then (url) =>
|
||||
@window.open(url, "_self")
|
||||
@window.open url, "_self"
|
||||
, (err) =>
|
||||
@confirm.notify('error', err)
|
||||
else if from == "github"
|
||||
callbackUri = @location.absUrl() + "/github"
|
||||
@githubService.getAuthUrl(callbackUri).then (url) =>
|
||||
|
|
|
@ -68,6 +68,13 @@ describe "ImportProjectCtrl", ->
|
|||
|
||||
$provide.value("$window", mocks.window)
|
||||
|
||||
_mockConfirm = ->
|
||||
mocks.confirm = {
|
||||
notify: sinon.stub()
|
||||
}
|
||||
|
||||
$provide.value("$tgConfirm", mocks.confirm)
|
||||
|
||||
_mockLocation = ->
|
||||
mocks.location = {
|
||||
search: sinon.stub()
|
||||
|
@ -98,6 +105,7 @@ describe "ImportProjectCtrl", ->
|
|||
_mockJiraImportService()
|
||||
_mockAsanaImportService()
|
||||
_mockWindow()
|
||||
_mockConfirm()
|
||||
_mockLocation()
|
||||
_mockTgNavUrls()
|
||||
_mockRouteParams()
|
||||
|
|
|
@ -41,10 +41,12 @@ class JiraImportService extends taiga.Service
|
|||
@resources.jiraImporter.importProject(@.url, @.token, name, description, projectId, userBindings, keepExternalReference, isPrivate, projectType, importerType)
|
||||
|
||||
getAuthUrl: (url) ->
|
||||
return new Promise (resolve) =>
|
||||
return new Promise (resolve, reject) =>
|
||||
@resources.jiraImporter.getAuthUrl(url).then (response) =>
|
||||
@.authUrl = response.data.url
|
||||
resolve(@.authUrl)
|
||||
, (err) =>
|
||||
reject(err.data._error_message)
|
||||
|
||||
authorize: () ->
|
||||
return new Promise (resolve, reject) =>
|
||||
|
|
Loading…
Reference in New Issue