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.

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

check if the path is valid #328

Merged
merged 1 commit into from
Dec 8, 2022
Merged

check if the path is valid #328

merged 1 commit into from
Dec 8, 2022

Conversation

jburel
Copy link
Member

@jburel jburel commented Nov 24, 2022

This PR handles the attempt to import with invalid characters in the path.
The files will no be added to the list of files to import

@jburel jburel requested a review from dominikl November 24, 2022 07:35
@jburel jburel closed this Nov 24, 2022
@jburel jburel reopened this Nov 24, 2022
@dominikl
Copy link
Member

Looks good, thanks @jburel 👍 Fixes #325 , #174 and #146 . Haven't tested it though. Is there a way to test it? I struggled to replicate these errors in the first place.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants