vikunja
/
vikunja
6
43
Fork 72
Code Issues 151 Pull Requests 17 Packages Releases 32 Activity

Return migration errors for Trello #2209

Closed
Elscrux wants to merge 2 commits from Elscrux/vikunja:fix/trello-migration-errors into main
pull from: Elscrux/vikunja:fix/trello-migration-errors
merge into: vikunja:main
Conversation 3 Commits 2 Files Changed 2 +6 -3
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 9b491aba5b - Show all commits

9
pkg/modules/migration/handler/listeners.go
View File

@ -68,19 +68,22 @@ func (s *MigrationListener) Handle(msg *message.Message) (err error) {
err = ms.Migrate(event.User)
if err != nil {
return
log.Errorf("[Migration] Failed to migrate %d from %s for user %d: %s", m.ID, event.MigratorKind, event.User.ID, err)
return err
}
err = migration.FinishMigration(m)
if err != nil {
return
log.Errorf("[Migration] Failed to finish migration %d from %s for user %d: %s", m.ID, event.MigratorKind, event.User.ID, err)
return err
}
err = notifications.Notify(event.User, &MigrationDoneNotification{
MigratorName: ms.Name(),
})
if err != nil {
return
log.Errorf("[Migration] Failed to notify migration %d from %s for user %d: %s", m.ID, event.MigratorKind, event.User.ID, err)
return err
}
log.Debugf("[Migration] Successfully done migration %d from %s for user %d", m.ID, event.MigratorKind, event.User.ID)