Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve error information when file operation fail #85

Closed
come-nc opened this issue Apr 11, 2022 · 1 comment · Fixed by #103
Closed

Improve error information when file operation fail #85

come-nc opened this issue Apr 11, 2022 · 1 comment · Fixed by #103
Assignees
@come-nc
Labels
4. to release Ready to be released and/or waiting for tests to finish
Milestone
1.0

Comments

@come-nc
Copy link
Collaborator

come-nc commented Apr 11, 2022

This will need changes on server side for IImportSource and IExportDestination.
All file related operations should not return bool but throw when failing and give information about the path and exact error.
@come-nc come-nc added the 1. to develop Accepted and waiting to be taken care of label Apr 11, 2022
@come-nc come-nc added this to the 0.1 milestone Apr 11, 2022
@come-nc come-nc self-assigned this Apr 11, 2022
@come-nc come-nc mentioned this issue Apr 12, 2022
Closed
2 tasks
@come-nc
Copy link
Collaborator Author

come-nc commented Apr 12, 2022 •
edited
Loading

Closed by

@Pytal Pytal linked a pull request Apr 13, 2022 that will close this issue
Use exceptions instead of booleans #103
Merged
1 task
@come-nc come-nc added 4. to release Ready to be released and/or waiting for tests to finish and removed 1. to develop Accepted and waiting to be taken care of labels Apr 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@come-nc come-nc

Labels
4. to release Ready to be released and/or waiting for tests to finish
Projects
None yet
Milestone
1.0
Development

Successfully merging a pull request may close this issue.

Use exceptions instead of booleans nextcloud/user_migration
1 participant
@come-nc