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

Add support for non case senstive title ids #45

Open
Zhaxxy opened this issue Jul 18, 2024 · 1 comment
Open

Add support for non case senstive title ids #45

Zhaxxy opened this issue Jul 18, 2024 · 1 comment

Comments

@Zhaxxy
Copy link

Zhaxxy commented Jul 18, 2024

Could you make it so that the case sensitivity of title ids does not matter, perhaps what you can do is check if the exact title id folder exists, if not, then check if the CAPTIAL letter title id exists then, if not check if lowercase title id exists, then finally, raise the error.

This will make the tool just a bit easier to use as the error message can be a bit vague especially to less tech savy people

@sudokoko
Copy link
Member

sudokoko commented Jul 18, 2024

This could be resolved by simply using String.ToUpper() on the user provided input rather than checking all variations of the Title ID since it shouldn't really ever be lower case to my knowledge

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

No branches or pull requests

2 participants