vikunja
/
vikunja
6
43
Fork 71
Code Issues 150 Pull Requests 16 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 +9 -6
Elscrux commented 2024-03-16 23:47:36 +00:00
Contributor
Copy Link

As mentioned here: https://community.vikunja.io/t/trello-import-issues/2110/22?u=elscrux
This should forward errors from inside the Trello migration which hopefully makes it easier to debug any potential errors occurring during the migration process.

As mentioned here: https://community.vikunja.io/t/trello-import-issues/2110/22?u=elscrux This should forward errors from inside the Trello migration which hopefully makes it easier to debug any potential errors occurring during the migration process.
Elscrux added 3 commits 2024-03-16 23:47:38 +00:00
continuous-integration/drone/pr Build is passing Details
360e0c7610 Attempt to fix broken link
continuous-integration/drone/pr Build is failing Details
3cb70b0f0c Return errors in Trello migration process
Elscrux force-pushed fix/trello-migration-errors from 3cb70b0f0c to 3c87b5cea7 2024-03-16 23:49:25 +00:00 Compare
frederick commented 2024-03-17 00:02:00 +00:00
Member
Copy Link

Hi Elscrux!

Thank you for creating a PR!

I've deployed the frontend changes of this PR on a preview environment under this URL: https://2209-fix-trello-migration-errors--vikunja-frontend-preview.netlify.app

You can use this url to view the changes live and test them out.
You will need to manually connect this to an api running somewhere. The easiest to use is https://try.vikunja.io/.

This preview does not contain any changes made to the api, only the frontend.

Have a nice day!

Beep boop, I'm a bot.

Hi Elscrux! Thank you for creating a PR! I've deployed the frontend changes of this PR on a preview environment under this URL: https://2209-fix-trello-migration-errors--vikunja-frontend-preview.netlify.app You can use this url to view the changes live and test them out. You will need to manually connect this to an api running somewhere. The easiest to use is https://try.vikunja.io/. This preview does not contain any changes made to the api, only the frontend. Have a nice day! > Beep boop, I'm a bot.
frederick commented 2024-03-17 00:02:00 +00:00
Member
Copy Link

Hi Elscrux!

Thank you for creating a PR!

I've deployed the frontend changes of this PR on a preview environment under this URL: https://2209-fix-trello-migration-errors--vikunja-frontend-preview.netlify.app

You can use this url to view the changes live and test them out.
You will need to manually connect this to an api running somewhere. The easiest to use is https://try.vikunja.io/.

This preview does not contain any changes made to the api, only the frontend.

Have a nice day!

Beep boop, I'm a bot.

Hi Elscrux! Thank you for creating a PR! I've deployed the frontend changes of this PR on a preview environment under this URL: https://2209-fix-trello-migration-errors--vikunja-frontend-preview.netlify.app You can use this url to view the changes live and test them out. You will need to manually connect this to an api running somewhere. The easiest to use is https://try.vikunja.io/. This preview does not contain any changes made to the api, only the frontend. Have a nice day! > Beep boop, I'm a bot.
Elscrux added 1 commit 2024-03-17 13:57:59 +00:00
continuous-integration/drone/pr Build is passing Details
9b491aba5b Add error messages
Elscrux commented 2024-03-17 14:34:05 +00:00
Author
Contributor
Copy Link

Nevermind, I just didn't seem to understand Go error handling

Nevermind, I just didn't seem to understand Go error handling
Elscrux closed this pull request 2024-03-17 14:34:05 +00:00
All checks were successful
continuous-integration/drone/pr Build is passing
Required
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
dependencies

   
duplicate

This issue or pull request already exists

  
help wanted

Need some help

  
invalid

Something is wrong

  
kind/bug

Something is not working

  
kind/feature

New feature

  
needs reproduction

   
question

More information is needed

  
security

   
wontfix

This won't be fixed

No Label
dependencies
duplicate
help wanted
invalid
kind/bug
kind/feature
needs reproduction
question
security
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: vikunja/vikunja#2209
No description provided.
Delete Branch "Elscrux/vikunja:fix/trello-migration-errors"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?