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

Make it more obvious that temp file path is required #5

Closed
linearray opened this Issue Jan 20, 2019 · 1 comment

Comments

Projects
None yet
2 participants
@linearray
Copy link

linearray commented Jan 20, 2019

When you do not enter a temp file path you just get the exception System.ArgumentException: The path is not of a legal form.

I almost didn't figure it out and thought there is something wrong with the input file path. Maybe it's best to just use System.IO.Path.GetTempPath() if the user doesn't enter anything?

@moisesmcardona

This comment has been minimized.

Copy link
Owner

moisesmcardona commented Jan 20, 2019

Totally forgot about checking the input, output and temporary fields. It has been fixed in the latest commits.

I'd prefer the user specifies another folder other than the system temporary files as that way it doesn't possibly mess with other temporary files in that folder.

The user will now receive a message telling him to browse for a temporary path

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment